Rockbox

Tasklist

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

Attached to Project: Rockbox
Opened by Kai Posadowsky (Riffer) - Thursday, 08 November 2012, 21:40 GMT
Last edited by Dominik Riebeling (bluebrother) - Friday, 09 November 2012, 20:17 GMT
Task Type Patches
Category Music playback
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Release 3.11
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

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.
This task depends upon

Closed by  Dominik Riebeling (bluebrother)
Friday, 09 November 2012, 20:17 GMT
Reason for closing:  Out of Date
Additional comments about closing:  Patch is now on gerrit: http://gerrit.rockbox.org/r/#/c/349/
Comment by Kai Posadowsky (Riffer) - Thursday, 08 November 2012, 21:41 GMT
here the patch itself
Comment by Dominik Riebeling (bluebrother) - Thursday, 08 November 2012, 22:05 GMT
We prefer patches on gerrit these days. Is it possible for you to push it there?
Comment by Kai Posadowsky (Riffer) - Friday, 09 November 2012, 19:17 GMT
After having a good time with gerrit (maybe the instructions should say very clearly that the Username is case sensitive?) its done:

http://gerrit.rockbox.org/r/#/c/349/


Comment by Kai Posadowsky (Riffer) - Friday, 09 November 2012, 20:13 GMT
(forget about the case sensitive, I just added it - nice to be part of the team!)

Someone could lease close this. Many thanks!
Comment by Kai Posadowsky (Riffer) - Friday, 09 November 2012, 20:14 GMT
*please

Loading...