[vlc-devel] [PATCH 1/6] Added a new button for playlist shuffle. So far the button invokes a dummy procedure.

Felix Paul Kühne fkuehne.videolan at gmail.com
Sat Mar 9 11:59:43 CET 2013


Hello,

First of all, you should squash some of the patches. For our git history, your preliminary implementation is not important, but only your result.

Regarding your implementation: if you dislike the current shuffle function (and there are many reasons to dislike it), please fix it in the playlist core and don't add hacks to the UI module. This way, all interface modules can benefit, like the http, console or Mac modules.

Best regards,

Felix


More information about the vlc-devel mailing list