[vlc-devel] [PATCH 1/2] core: added sort by date (prepare for playlist fix)

[Marcel Schnirring]

On Wed, Dec 21, 2016 at 9:25 PM, Rémi Denis-Courmont <remi at remlab.net>
wrote:

> Hello,
>
> Le keskiviikkona 21. joulukuuta 2016, 21.10.35 EET Marcel Schnirring a
> écrit :
> >  SORTFN( SORT_ARTIST, first, second )
> >  {
> >      int i_ret = meta_sort( first, second, vlc_meta_Artist, false );
> > -    /* Items came from the same artist: compare the albums */
> > +    /* Items came from the same artist: compare the dates */
> >      if( i_ret == 0 )
> > -        i_ret = proto_SORT_ALBUM( first, second );
> > +        i_ret = proto_SORT_DATE( first, second );
> >
> >      return i_ret;
> >  }
>
> This might be an improvement, but it does not seem to match with the commit
> message?
>

I added the new feature in the corresponding place in the chain:
Previously: Artist -> Album
Currently: Artist -> Date -> Album

*Mit freundlichen Grüßen / Kind regards,*
Marcel Schnirring
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20161221/d02c8cf9/attachment.html>