[vlc-devel] [vlc-commits] core: playlist: implement sorting
Zhao Zhili
quinkblack at foxmail.com
Tue Nov 20 04:16:16 CET 2018
Hi Romain,
Would you mind implementing a compatibility fallback for qsort_r()? It's
not available on Android.
error: implicit declaration of function 'qsort_r' is invalid in C99
[-Werror,-Wimplicit-function-declaration]
On 2018/11/15 下午11:23, Romain Vimont wrote:
> vlc | branch: master | Romain Vimont <rom1v at videolabs.io> | Tue Oct 16 15:51:02 2018 +0200| [a6832090ec73e28f71729be9d58af5c769f9a7a6] | committer: Thomas Guillem
>
> core: playlist: implement sorting
>
> Expose a function to sort the items in the playlist by a list of
> criteria. A criterion is composed of a key (title, duration, etc.) and
> an order (ascending or descending).
>
> Signed-off-by: Thomas Guillem <thomas at gllm.fr>
>
>> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=a6832090ec73e28f71729be9d58af5c769f9a7a6
> ---
>
> include/vlc_fixups.h | 4 +
> include/vlc_playlist.h | 40 +++++
> src/Makefile.am | 4 +-
> src/libvlccore.sym | 1 +
> src/playlist/sort.c | 404 +++++++++++++++++++++++++++++++++++++++++++++++++
> src/playlist/test.c | 108 +++++++++++++
> 6 files changed, 560 insertions(+), 1 deletion(-)
>
> diff --git a/include/vlc_fixups.h b/include/vlc_fixups.h
> index caf1768f63..598521c510 100644
> --- a/include/vlc_fixups.h
> +++ b/include/vlc_fixups.h
> @@ -112,6 +112,10 @@ typedef struct
> # include <dirent.h>
> #endif
>
> +#ifdef _WIN32
> +# define qsort_r qsort_s
> +#endif
> +
> #ifdef __cplusplus
> # define VLC_NOTHROW throw ()
> extern "C" {
> diff --git a/include/vlc_playlist.h b/include/vlc_playlist.h
> index c9074912b3..9253bbb90d 100644
> --- a/include/vlc_playlist.h
> +++ b/include/vlc_playlist.h
> @@ -126,6 +126,33 @@ enum vlc_playlist_playback_order
> VLC_PLAYLIST_PLAYBACK_ORDER_RANDOM,
> };
>
> +enum vlc_playlist_sort_key
> +{
> + VLC_PLAYLIST_SORT_KEY_TITLE,
> + VLC_PLAYLIST_SORT_KEY_DURATION,
> + VLC_PLAYLIST_SORT_KEY_ARTIST,
> + VLC_PLAYLIST_SORT_KEY_ALBUM,
> + VLC_PLAYLIST_SORT_KEY_ALBUM_ARTIST,
> + VLC_PLAYLIST_SORT_KEY_GENRE,
> + VLC_PLAYLIST_SORT_KEY_DATE,
> + VLC_PLAYLIST_SORT_KEY_TRACK_NUMBER,
> + VLC_PLAYLIST_SORT_KEY_DISC_NUMBER,
> + VLC_PLAYLIST_SORT_KEY_URL,
> + VLC_PLAYLIST_SORT_KEY_RATING,
> +};
> +
> +enum vlc_playlist_sort_order
> +{
> + VLC_PLAYLIST_SORT_ORDER_ASCENDING,
> + VLC_PLAYLIST_SORT_ORDER_DESCENDING,
> +};
> +
> +struct vlc_playlist_sort_criterion
> +{
> + enum vlc_playlist_sort_key key;
> + enum vlc_playlist_sort_order order;
> +};
> +
> /**
> * Playlist callbacks.
> *
> @@ -590,6 +617,19 @@ VLC_API void
> vlc_playlist_Shuffle(vlc_playlist_t *playlist);
>
> /**
> + * Sort the playlist by a list of criteria.
> + *
> + * \param playlist the playlist, locked
> + * \param criteria the sort criteria (in order)
> + * \param count the number of criteria
> + * \return VLC_SUCCESS on success, another value on error
> + */
> +VLC_API int
> +vlc_playlist_Sort(vlc_playlist_t *playlist,
> + const struct vlc_playlist_sort_criterion criteria[],
> + size_t count);
> +
> +/**
> * Return the index of a given item.
> *
> * \param playlist the playlist, locked
> diff --git a/src/Makefile.am b/src/Makefile.am
> index 58cc5a3319..d6bf427179 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -245,6 +245,7 @@ libvlccore_la_SOURCES = \
> playlist/randomizer.h \
> playlist/request.c \
> playlist/shuffle.c \
> + playlist/sort.c \
> preparser/art.c \
> preparser/art.h \
> preparser/fetcher.c \
> @@ -595,7 +596,8 @@ test_playlist_SOURCES = playlist/test.c \
> playlist/preparse.c \
> playlist/randomizer.c \
> playlist/request.c \
> - playlist/shuffle.c
> + playlist/shuffle.c \
> + playlist/sort.c
> test_playlist_CFLAGS = -DTEST_PLAYLIST
> test_randomizer_SOURCES = playlist/randomizer.c
> test_randomizer_CFLAGS = -DTEST_RANDOMIZER
> diff --git a/src/libvlccore.sym b/src/libvlccore.sym
> index 52c8017a3a..3a88031d21 100644
> --- a/src/libvlccore.sym
> +++ b/src/libvlccore.sym
> @@ -926,6 +926,7 @@ vlc_playlist_RequestInsert
> vlc_playlist_RequestMove
> vlc_playlist_RequestRemove
> vlc_playlist_Shuffle
> +vlc_playlist_Sort
> vlc_playlist_IndexOf
> vlc_playlist_IndexOfMedia
> vlc_playlist_GetPlaybackRepeat
> diff --git a/src/playlist/sort.c b/src/playlist/sort.c
> new file mode 100644
> index 0000000000..b44f81a168
> --- /dev/null
> +++ b/src/playlist/sort.c
> @@ -0,0 +1,404 @@
> +/*****************************************************************************
> + * playlist/sort.c
> + *****************************************************************************
> + * Copyright (C) 2018 VLC authors and VideoLAN
> + *
> + * This program is free software; you can redistribute it and/or modify it
> + * under the terms of the GNU Lesser General Public License as published by
> + * the Free Software Foundation; either version 2.1 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU Lesser General Public License for more details.
> + *
> + * You should have received a copy of the GNU Lesser General Public License
> + * along with this program; if not, write to the Free Software Foundation,
> + * Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
> + *****************************************************************************/
> +
> +#ifdef HAVE_CONFIG_H
> +# include "config.h"
> +#endif
> +
> +#include <vlc_common.h>
> +#include <vlc_rand.h>
> +#include "control.h"
> +#include "item.h"
> +#include "notify.h"
> +#include "playlist.h"
> +
> +/**
> + * Struct containing a copy of (parsed) media metadata, used for sorting
> + * without locking all the items.
> + */
> +struct vlc_playlist_item_meta {
> + vlc_playlist_item_t *item;
> + const char *title_or_name;
> + vlc_tick_t duration;
> + const char *artist;
> + const char *album;
> + const char *album_artist;
> + const char *genre;
> + const char *url;
> + int64_t date;
> + int64_t track_number;
> + int64_t disc_number;
> + int64_t rating;
> + bool has_date;
> + bool has_track_number;
> + bool has_disc_number;
> + bool has_rating;
> +};
> +
> +static int
> +vlc_playlist_item_meta_CopyString(const char **to, const char *from)
> +{
> + if (from)
> + {
> + *to = strdup(from);
> + if (unlikely(!*to))
> + return VLC_ENOMEM;
> + }
> + else
> + *to = NULL;
> + return VLC_SUCCESS;
> +}
> +
> +static int
> +vlc_playlist_item_meta_InitField(struct vlc_playlist_item_meta *meta,
> + enum vlc_playlist_sort_key key)
> +{
> + input_item_t *media = meta->item->media;
> + switch (key)
> + {
> + case VLC_PLAYLIST_SORT_KEY_TITLE:
> + {
> + const char *value = input_item_GetMetaLocked(media, vlc_meta_Title);
> + if (EMPTY_STR(value))
> + value = media->psz_name;
> + return vlc_playlist_item_meta_CopyString(&meta->title_or_name,
> + value);
> + }
> + case VLC_PLAYLIST_SORT_KEY_DURATION:
> + {
> + if (media->i_duration == INPUT_DURATION_INDEFINITE
> + || media->i_duration == INPUT_DURATION_UNSET)
> + meta->duration = 0;
> + else
> + meta->duration = media->i_duration;
> + return VLC_SUCCESS;
> + }
> + case VLC_PLAYLIST_SORT_KEY_ARTIST:
> + {
> + const char *value = input_item_GetMetaLocked(media,
> + vlc_meta_Artist);
> + return vlc_playlist_item_meta_CopyString(&meta->artist, value);
> + }
> + case VLC_PLAYLIST_SORT_KEY_ALBUM:
> + {
> + const char *value = input_item_GetMetaLocked(media, vlc_meta_Album);
> + return vlc_playlist_item_meta_CopyString(&meta->album, value);
> + }
> + case VLC_PLAYLIST_SORT_KEY_ALBUM_ARTIST:
> + {
> + const char *value = input_item_GetMetaLocked(media,
> + vlc_meta_AlbumArtist);
> + return vlc_playlist_item_meta_CopyString(&meta->album_artist,
> + value);
> + }
> + case VLC_PLAYLIST_SORT_KEY_GENRE:
> + {
> + const char *value = input_item_GetMetaLocked(media, vlc_meta_Genre);
> + return vlc_playlist_item_meta_CopyString(&meta->genre, value);
> + }
> + case VLC_PLAYLIST_SORT_KEY_DATE:
> + {
> + const char *str = input_item_GetMetaLocked(media, vlc_meta_Date);
> + meta->has_date = !EMPTY_STR(str);
> + if (meta->has_date)
> + meta->date = atoll(str);
> + return VLC_SUCCESS;
> + }
> + case VLC_PLAYLIST_SORT_KEY_TRACK_NUMBER:
> + {
> + const char *str = input_item_GetMetaLocked(media,
> + vlc_meta_TrackNumber);
> + meta->has_track_number = !EMPTY_STR(str);
> + if (meta->has_track_number)
> + meta->track_number = atoll(str);
> + return VLC_SUCCESS;
> + }
> + case VLC_PLAYLIST_SORT_KEY_DISC_NUMBER:
> + {
> + const char *str = input_item_GetMetaLocked(media,
> + vlc_meta_DiscNumber);
> + meta->has_disc_number = !EMPTY_STR(str);
> + if (meta->has_disc_number)
> + meta->disc_number = atoll(str);
> + return VLC_SUCCESS;
> + }
> + case VLC_PLAYLIST_SORT_KEY_URL:
> + {
> + const char *value = input_item_GetMetaLocked(media, vlc_meta_URL);
> + return vlc_playlist_item_meta_CopyString(&meta->url, value);
> + }
> + case VLC_PLAYLIST_SORT_KEY_RATING:
> + {
> + const char *str = input_item_GetMetaLocked(media, vlc_meta_Rating);
> + meta->has_rating = !EMPTY_STR(str);
> + if (meta->has_rating)
> + meta->rating = atoll(str);
> + return VLC_SUCCESS;
> + }
> + default:
> + assert(!"Unknown sort key");
> + vlc_assert_unreachable();
> + }
> +}
> +
> +static void
> +vlc_playlist_item_meta_DestroyFields(struct vlc_playlist_item_meta *meta)
> +{
> + free((void *) meta->title_or_name);
> + free((void *) meta->artist);
> + free((void *) meta->album);
> + free((void *) meta->album_artist);
> + free((void *) meta->genre);
> + free((void *) meta->url);
> +}
> +
> +static int
> +vlc_playlist_item_meta_InitFields(struct vlc_playlist_item_meta *meta,
> + const struct vlc_playlist_sort_criterion criteria[], size_t count)
> +{
> + for (size_t i = 0; i < count; ++i)
> + {
> + const struct vlc_playlist_sort_criterion *criterion = &criteria[i];
> + int ret = vlc_playlist_item_meta_InitField(meta, criterion->key);
> + if (unlikely(ret != VLC_SUCCESS))
> + {
> + vlc_playlist_item_meta_DestroyFields(meta);
> + return ret;
> + }
> + }
> + return VLC_SUCCESS;
> +}
> +
> +static struct vlc_playlist_item_meta *
> +vlc_playlist_item_meta_New(vlc_playlist_item_t *item,
> + const struct vlc_playlist_sort_criterion criteria[],
> + size_t count)
> +{
> + /* assume that NULL representation is all-zeros */
> + struct vlc_playlist_item_meta *meta = calloc(1, sizeof(*meta));
> + if (unlikely(!meta))
> + return NULL;
> +
> + meta->item = item;
> +
> + vlc_mutex_lock(&item->media->lock);
> + int ret = vlc_playlist_item_meta_InitFields(meta, criteria, count);
> + vlc_mutex_unlock(&item->media->lock);
> +
> + if (unlikely(ret != VLC_SUCCESS))
> + {
> + free(meta);
> + return NULL;
> + }
> +
> + return meta;
> +}
> +
> +static void
> +vlc_playlist_item_meta_Delete(struct vlc_playlist_item_meta *meta)
> +{
> + vlc_playlist_item_meta_DestroyFields(meta);
> + free(meta);
> +}
> +
> +static inline int
> +CompareStrings(const char *a, const char *b)
> +{
> + if (a && b)
> + return strcasecmp(a, b);
> + if (!a && !b)
> + return 0;
> + return a ? 1 : -1;
> +}
> +
> +static inline int
> +CompareIntegers(int64_t a, int64_t b)
> +{
> + if (a < b)
> + return -1;
> + if (a > b)
> + return 1;
> + return 0;
> +}
> +
> +static inline int
> +CompareOptionalIntegers(bool has_a, int64_t a, bool has_b, int64_t b)
> +{
> + if (has_a && has_b)
> + return CompareIntegers(a, b);
> +
> + if (!has_a && !has_b)
> + return 0;
> +
> + return a ? 1 : -1;
> +}
> +
> +static inline int
> +CompareMetaByKey(const struct vlc_playlist_item_meta *a,
> + const struct vlc_playlist_item_meta *b,
> + enum vlc_playlist_sort_key key)
> +{
> + switch (key)
> + {
> + case VLC_PLAYLIST_SORT_KEY_TITLE:
> + return CompareStrings(a->title_or_name, b->title_or_name);
> + case VLC_PLAYLIST_SORT_KEY_DURATION:
> + return CompareIntegers(a->duration, b->duration);
> + case VLC_PLAYLIST_SORT_KEY_ARTIST:
> + return CompareStrings(a->artist, b->artist);
> + case VLC_PLAYLIST_SORT_KEY_ALBUM:
> + return CompareStrings(a->album, b->album);
> + case VLC_PLAYLIST_SORT_KEY_ALBUM_ARTIST:
> + return CompareStrings(a->album_artist, b->album_artist);
> + case VLC_PLAYLIST_SORT_KEY_GENRE:
> + return CompareStrings(a->genre, b->genre);
> + case VLC_PLAYLIST_SORT_KEY_DATE:
> + return CompareOptionalIntegers(a->has_date, a->date,
> + b->has_date, b->date);
> + case VLC_PLAYLIST_SORT_KEY_TRACK_NUMBER:
> + return CompareOptionalIntegers(a->has_track_number, a->track_number,
> + b->has_track_number, b->track_number);
> + case VLC_PLAYLIST_SORT_KEY_DISC_NUMBER:
> + return CompareOptionalIntegers(a->has_disc_number, a->disc_number,
> + b->has_disc_number, b->disc_number);
> + case VLC_PLAYLIST_SORT_KEY_URL:
> + return CompareStrings(a->url, b->url);
> + case VLC_PLAYLIST_SORT_KEY_RATING:
> + return CompareOptionalIntegers(a->has_rating, a->rating,
> + b->has_rating, b->rating);
> + default:
> + assert(!"Unknown sort key");
> + vlc_assert_unreachable();
> + }
> +}
> +
> +/* context for qsort_r() */
> +struct sort_request
> +{
> + const struct vlc_playlist_sort_criterion *criteria;
> + size_t count;
> +};
> +
> +static int
> +compare_meta(const void *lhs, const void *rhs, void *userdata)
> +{
> + struct sort_request *req = userdata;
> + const struct vlc_playlist_item_meta *a =
> + *(const struct vlc_playlist_item_meta **) lhs;
> + const struct vlc_playlist_item_meta *b =
> + *(const struct vlc_playlist_item_meta **) rhs;
> +
> + for (size_t i = 0; i < req->count; ++i)
> + {
> + const struct vlc_playlist_sort_criterion *criterion = &req->criteria[i];
> + int ret = CompareMetaByKey(a, b, criterion->key);
> + if (ret)
> + {
> + if (criterion->order == VLC_PLAYLIST_SORT_ORDER_DESCENDING)
> + /* do not return -ret, it's undefined if ret == INT_MIN */
> + return ret > 0 ? -1 : 1;
> + return ret;
> + }
> + }
> + return 0;
> +}
> +
> +static void
> +vlc_playlist_DeleteMetaArray(struct vlc_playlist_item_meta *array[],
> + size_t count)
> +{
> + for (size_t i = 0; i < count; ++i)
> + vlc_playlist_item_meta_Delete(array[i]);
> + free(array);
> +}
> +
> +static struct vlc_playlist_item_meta **
> +vlc_playlist_NewMetaArray(vlc_playlist_t *playlist,
> + const struct vlc_playlist_sort_criterion criteria[], size_t count)
> +{
> + struct vlc_playlist_item_meta **array =
> + vlc_alloc(playlist->items.size, sizeof(*array));
> +
> + if (unlikely(!array))
> + return NULL;
> +
> + size_t i;
> + for (i = 0; i < playlist->items.size; ++i)
> + {
> + array[i] = vlc_playlist_item_meta_New(playlist->items.data[i],
> + criteria, count);
> + if (unlikely(!array[i]))
> + break;
> + }
> +
> + if (i < playlist->items.size)
> + {
> + /* allocation failure */
> + vlc_playlist_DeleteMetaArray(array, i);
> + return NULL;
> + }
> +
> + return array;
> +}
> +
> +int
> +vlc_playlist_Sort(vlc_playlist_t *playlist,
> + const struct vlc_playlist_sort_criterion criteria[],
> + size_t count)
> +{
> + assert(count > 0);
> + vlc_playlist_AssertLocked(playlist);
> +
> + vlc_playlist_item_t *current = playlist->current != -1
> + ? playlist->items.data[playlist->current]
> + : NULL;
> +
> + struct vlc_playlist_item_meta **array =
> + vlc_playlist_NewMetaArray(playlist, criteria, count);
> + if (unlikely(!array))
> + return VLC_ENOMEM;
> +
> + struct sort_request req = { criteria, count };
> +
> + qsort_r(array, playlist->items.size, sizeof(*array), compare_meta, &req);
> +
> + /* apply the sorting result to the playlist */
> + for (size_t i = 0; i < playlist->items.size; ++i)
> + playlist->items.data[i] = array[i]->item;
> +
> + vlc_playlist_DeleteMetaArray(array, playlist->items.size);
> +
> + struct vlc_playlist_state state;
> + if (current)
> + {
> + /* the current position have changed after the shuffle */
> + vlc_playlist_state_Save(playlist, &state);
> + playlist->current = vlc_playlist_IndexOf(playlist, current);
> + playlist->has_prev = vlc_playlist_ComputeHasPrev(playlist);
> + playlist->has_next = vlc_playlist_ComputeHasNext(playlist);
> + }
> +
> + vlc_playlist_Notify(playlist, on_items_reset, playlist->items.data,
> + playlist->items.size);
> + if (current)
> + vlc_playlist_state_NotifyChanges(playlist, &state);
> +
> + return VLC_SUCCESS;
> +}
> diff --git a/src/playlist/test.c b/src/playlist/test.c
> index 888e76efa9..5c3b75f3be 100644
> --- a/src/playlist/test.c
> +++ b/src/playlist/test.c
> @@ -2077,6 +2077,113 @@ test_shuffle(void)
> vlc_playlist_Delete(playlist);
> }
>
> +static void
> +test_sort(void)
> +{
> + vlc_playlist_t *playlist = vlc_playlist_New(NULL);
> + assert(playlist);
> +
> + input_item_t *media[10];
> + media[0] = CreateDummyMedia(4); media[0]->i_duration = 42;
> + media[1] = CreateDummyMedia(1); media[1]->i_duration = 5;
> + media[2] = CreateDummyMedia(6); media[2]->i_duration = 100;
> + media[3] = CreateDummyMedia(2); media[3]->i_duration = 1;
> + media[4] = CreateDummyMedia(1); media[4]->i_duration = 8;
> + media[5] = CreateDummyMedia(4); media[5]->i_duration = 23;
> + media[6] = CreateDummyMedia(3); media[6]->i_duration = 60;
> + media[7] = CreateDummyMedia(3); media[7]->i_duration = 40;
> + media[8] = CreateDummyMedia(0); media[8]->i_duration = 42;
> + media[9] = CreateDummyMedia(5); media[9]->i_duration = 42;
> +
> + /* initial playlist with 10 items */
> + int ret = vlc_playlist_Append(playlist, media, 10);
> + assert(ret == VLC_SUCCESS);
> +
> + struct vlc_playlist_callbacks cbs = {
> + .on_items_reset = callback_on_items_reset,
> + .on_current_index_changed = callback_on_current_index_changed,
> + .on_has_prev_changed = callback_on_has_prev_changed,
> + .on_has_next_changed = callback_on_has_next_changed,
> + };
> +
> + struct callback_ctx ctx = CALLBACK_CTX_INITIALIZER;
> + vlc_playlist_listener_id *listener =
> + vlc_playlist_AddListener(playlist, &cbs, &ctx);
> + assert(listener);
> +
> + /* on_items_reset is called once during AddListener() */
> + callback_ctx_reset(&ctx);
> +
> + playlist->current = 0;
> + playlist->has_prev = false;
> + playlist->has_next = true;
> +
> + struct vlc_playlist_sort_criterion criteria1[] = {
> + { VLC_PLAYLIST_SORT_KEY_TITLE, VLC_PLAYLIST_SORT_ORDER_ASCENDING },
> + { VLC_PLAYLIST_SORT_KEY_DURATION, VLC_PLAYLIST_SORT_ORDER_ASCENDING },
> + };
> + vlc_playlist_Sort(playlist, criteria1, 2);
> +
> + EXPECT_AT(0, 8);
> + EXPECT_AT(1, 1);
> + EXPECT_AT(2, 4);
> + EXPECT_AT(3, 3);
> + EXPECT_AT(4, 7);
> + EXPECT_AT(5, 6);
> + EXPECT_AT(6, 5);
> + EXPECT_AT(7, 0);
> + EXPECT_AT(8, 9);
> + EXPECT_AT(9, 2);
> +
> + ssize_t index = vlc_playlist_IndexOfMedia(playlist, media[0]);
> + assert(index == 7);
> + assert(playlist->current == 7);
> +
> + assert(ctx.vec_items_reset.size == 1);
> + assert(ctx.vec_items_reset.data[0].count == 10);
> + assert(ctx.vec_items_reset.data[0].state.playlist_size == 10);
> + assert(ctx.vec_items_reset.data[0].state.current == 7);
> + assert(ctx.vec_items_reset.data[0].state.has_prev);
> + assert(ctx.vec_items_reset.data[0].state.has_next);
> +
> + assert(ctx.vec_current_index_changed.size == 1);
> + assert(ctx.vec_current_index_changed.data[0].current == 7);
> +
> + assert(ctx.vec_has_prev_changed.size == 1);
> + assert(ctx.vec_has_prev_changed.data[0].has_prev);
> +
> + assert(ctx.vec_has_next_changed.size == 0);
> +
> + callback_ctx_reset(&ctx);
> +
> + struct vlc_playlist_sort_criterion criteria2[] = {
> + { VLC_PLAYLIST_SORT_KEY_DURATION, VLC_PLAYLIST_SORT_ORDER_DESCENDING },
> + { VLC_PLAYLIST_SORT_KEY_TITLE, VLC_PLAYLIST_SORT_ORDER_ASCENDING },
> + };
> +
> + vlc_playlist_Sort(playlist, criteria2, 2);
> +
> + EXPECT_AT(0, 2);
> + EXPECT_AT(1, 6);
> + EXPECT_AT(2, 8);
> + EXPECT_AT(3, 0);
> + EXPECT_AT(4, 9);
> + EXPECT_AT(5, 7);
> + EXPECT_AT(6, 5);
> + EXPECT_AT(7, 4);
> + EXPECT_AT(8, 1);
> + EXPECT_AT(9, 3);
> +
> + assert(ctx.vec_items_reset.size == 1);
> + assert(ctx.vec_items_reset.data[0].count == 10);
> + assert(ctx.vec_items_reset.data[0].state.playlist_size == 10);
> +
> + callback_ctx_destroy(&ctx);
> + vlc_playlist_RemoveListener(playlist, listener);
> + DestroyMediaArray(media, 10);
> + vlc_playlist_Delete(playlist);
> +}
> +
> #undef EXPECT_AT
>
> int main(void)
> @@ -2109,5 +2216,6 @@ int main(void)
> test_request_goto_adapt();
> test_random();
> test_shuffle();
> + test_sort();
> return 0;
> }
>
> _______________________________________________
> vlc-commits mailing list
> vlc-commits at videolan.org
> https://mailman.videolan.org/listinfo/vlc-commits
More information about the vlc-devel
mailing list