Review Request: Fix duplicate name bug in creating playlist folders

Commit Hook null at kde.org
Wed Jan 9 15:39:01 UTC 2013


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


This review has been submitted with commit a358770be9650802ab2d7e780019bc502e4cbaa6 by Edward Hades on behalf of Peter Amidon to branch master.

- Commit Hook


On Jan. 8, 2013, 3:57 p.m., Andromeda Galaxy wrote:
> 
> -----------------------------------------------------------
> 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.
> 
> 
> Description
> -------
> 
> This patch fixes a problem where folder-names were lexiographically sorted to find the largest number of duplicates
> 
> 
> Diffs
> -----
> 
>   src/browsers/playlistbrowser/PlaylistBrowserCategory.cpp f6f06d6 
> 
> Diff: http://git.reviewboard.kde.org/r/108245/diff/
> 
> 
> Testing
> -------
> 
> 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/20130109/73443132/attachment.html>


More information about the Amarok-devel mailing list