• Status Closed
  • Percent Complete
  • Task Type Patches
  • Category Music playback
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Release 3.11
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Riffer - 2012-11-08
Last edited by bluebrother - 2012-11-09

FS#12779 - This patch reworks the sleep timer a little bit and fixes a small bug.

After seeing that nearly all what I wanted to have regarding the sleep timer I had only one point left: The sleep timer should not shut down but instead turn into pause playing. After this the standard poweroff timer gets to work. If the poweroff timer is not set the sleep timer directly shuts down as before.

This fixes a bug too - there was a pause on sleep timer end but only for the situation where usb (I think mostly for battery purpose?) is connected. But after that the sleep timer won’t be restarted on button use! This has been fixed now.

Closed by  bluebrother
2012-11-09 20:17
Reason for closing:  Out of Date
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Patch is now on gerrit: /r/#/c/349/

here the patch itself

We prefer patches on gerrit these days. Is it possible for you to push it there?

After having a good time with gerrit (maybe the instructions should say very clearly that the Username is case sensitive?) its done:

(forget about the case sensitive, I just added it - nice to be part of the team!)

Someone could lease close this. Many thanks!



Available keyboard shortcuts


Task Details

Task Editing