Review Request 114634: Make playlist switching code simpler
Shantanu Tushar
shantanu at kde.org
Mon Dec 23 19:17:21 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/114634/
-----------------------------------------------------------
Review request for Plasma and Sinny Kumari.
Repository: plasma-mediacenter
Description
-------
Separate methods for switching and set'ing a new playlist aren't really necessary. This patch eliminates the duplication, and also introduces some more cleanup to related methods.
Diffs
-----
libs/mediacenter/playlistmodel.h 5a4ac8c
libs/mediacenter/playlistmodel.cpp cac6154
libs/mediacenter/multipleplaylistmodel.cpp 10ccb40
Diff: https://git.reviewboard.kde.org/r/114634/diff/
Testing
-------
Adding/removing playlist, adding/removing media, removing default playlist work fine.
Thanks,
Shantanu Tushar
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20131223/c4d96147/attachment.html>
More information about the Plasma-devel
mailing list