FS#7911 - Decouple playlist editting from the play/stop state
This patch decouples playlist editting from the play/stop state. So,
you get the same playlist menu independent of whether rockbox is
playing or stopped (and all the commands do the same things). Also,
editting the playlist while stopped does not restart playback.
The idea behind this is that now the current playlist appears to
the user to be 'still there' when stopped (ie you can view it and
resume it), it doesn't make sense for an insert while stopped to
insert to an empty playlist.
I have done two patches, the first just makes the current playlist
menu available when stopped (no changes to the menu). The only changes
are to onplay.c and most of the changes were deletes, so the new code
is ~30 lines shorter than the original.
The second patch does the same as the first, but also:
1) it renames "Play Next" to "Replace"
2) it adds a "Replace shuffled" command
3) it renames "Insert" to "Insert in order" (and the same for queue)
4) it does not show the "Insert in order" order command until it is
useful: after an "Insert next" has already been done.
I think these changes make the playlist editing much easier to
understand for people new to rockbox (and easier to remember for those
not so new). In particular, the current "Insert" can do one of 3 different things (add to empty, insert next or insert after previous insert/ insert next). With the second patch all commands always do the same thing, and the renaming makes it a bit clearer what that thing is.
Anyone who wants to test this should probably also install http://www.rockbox.org/tracker/task/7898 - which fixes a bug in the "Insert next" code.