Review Request: Fix duplicate name bug in creating playlist folders

Andromeda Galaxy peter at picnicpark.org
Tue Jan 8 15:57:58 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/108245/
-----------------------------------------------------------

(Updated Jan. 8, 2013, 3:57 p.m.)


Review request for Amarok.


Changes
-------

Updated diff to use algorithm not requiring std::algorithm.


Description
-------

This patch fixes a problem where folder-names were lexiographically sorted to find the largest number of duplicates


Diffs (updated)
-----

  src/browsers/playlistbrowser/PlaylistBrowserCategory.cpp f6f06d6 

Diff: http://git.reviewboard.kde.org/r/108245/diff/


Testing (updated)
-------

This patch appears to fix the problems in Amarok's ui once compiled; there is no change in the unit test output.


Thanks,

Andromeda Galaxy

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20130108/b4164cd7/attachment.html>


More information about the Amarok-devel mailing list