[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
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.
More information about the vlc-devel