• Status Unconfirmed
  • Percent Complete
  • Task Type Patches
  • Category Playlists
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by wpyh - 2008-10-05

FS#9456 - Remove Repeat:Shuffle option and always reshuffle when Shuffle:Yes and Repeat:All

This patch removes the Repeat:Shuffle setting, and always reshuffles the playlist when Shuffle is set to yes and Repeat is set to All.

The rationale behind this is that a user would rarely want to shuffle the playlist the first time only and always repeat the same sequence afterwards. Rather, one would either play the playlist in order and repeat in the same order, or shuffle the playlist first and always reshuffle.

Setting combinations affected:
- Shuffle:Yes && Repeat:All

before: shuffle the playlist but repeats the same sequence
after: shuffle the playlist and reshuffles when the end is reached

- Shuffle:Yes && Repeat:Shuffle

before: shuffle the playlist and reshuffles when the end is reached
after: this combination is replaced by Shuffle:Yes && Repeat:All

- Shuffle:No && Repeat:Shuffle

before: reshuffle when the end is reached
after: rarely used combination, not available anymore

All other combinations are not affected.

The patch contains some comment fixes and some code simplification as well. We also have to handle a special case in the patch.

wpyh commented on 2008-10-10 18:16

Sorry, the previous patch doesn't apply cleany. Please try this new patch.

patch applies but doesnt compile

wpyh commented on 2008-12-25 14:25

This is a new patch sync'ed to the latest svn, and I removed a code change that tests for CONFIG_CODEC.

jdgordon: Sorry for not updating the status. The patch compiles cleanly on my side. Could you please try the new patch? If it still fails to compile, please post the target you are building for and the error message here.

wpyh commented on 2009-05-10 03:10

Sync'ed patch to latest SVN.

wpyh commented on 2009-05-17 15:38

Per discussion on IRC, now I would like to submit a smaller patch instead: Make Shuffle:Yes and Repeat:All behave the same as Repeat:Shuffle. This means that if Shuffle:Yes and Repeat:All, then at the end of the playlist, the playlist will be reshuffled.


Available keyboard shortcuts


Task Details

Task Editing