[amarok] /: Cleaned up playlist multilevel sorting algorithm. Added album artist special case to playlist sorting.

Matěj Laitl matej at laitl.cz
Sat Jun 22 19:07:06 UTC 2013


On 20. 6. 2013 Myriam Schweingruber wrote:
> On Sat, Apr 27, 2013 at 11:11 AM, Matěj Laitl <matej at laitl.cz> wrote:
> > Git commit 583158f4c4d25098aa1f0bb6b683813488580d52 by Matěj Laitl, on
> > behalf of Konrad Zemek. Committed on 27/04/2013 at 11:08.
> > Pushed by laitl into branch 'master'.
> > 
> > Cleaned up playlist multilevel sorting algorithm. Added album artist
> > special case to playlist sorting.
> > 
> > It fixes an issue with two albums being merged on the playlist when they
> > happen to have the same name, and sorting by album is enabled. Originally
> > thought to fix bug 271105, it actually fixes some other, but connected,
> > one (see my comment https://bugs.kde.org/show_bug.cgi?id=271105#c10 ).
> > 
> > Thanks go to Konrad Zemek for the patch.
> > 
> > REVIEW: 110070
> > CCBUG: 271105
> 
> Shouldn't this be added to the ChangeLog?

I think the user impact is rather small and hard to describe, so I wouldn't 
require adding it to ChangeLog.

Myriam, I suggest you bottom-post instead of top-posting (and quote just the 
relevant lines), that makes technical e-mail threads easier to read.

	Matěj


More information about the Amarok-devel mailing list