Re: Fix for default sleep timer duration as a shortcut

2012-10-06 Thread Richard Quirk
On 10/06/2012 03:22 PM, Dominik Riebeling wrote: On Sat, Oct 6, 2012 at 2:16 PM, Richard Quirk wrote: Thanks for reviewing that. Unfortunately I missed a set_sleep_timer in 327, but did not spot the compiler output as I had 327 and 328 staged on the same branch. Clean up here http://gerrit.rock

Re: Fix for default sleep timer duration as a shortcut

2012-10-06 Thread Dominik Riebeling
On Sat, Oct 6, 2012 at 2:16 PM, Richard Quirk wrote: > Thanks for reviewing that. Unfortunately I missed a set_sleep_timer in 327, > but did not spot the compiler output as I had 327 and 328 staged on the same > branch. Clean up here http://gerrit.rockbox.org/329 :-( Too late, since this broke pr

Re: Fix for default sleep timer duration as a shortcut

2012-10-06 Thread Richard Quirk
On 10/06/2012 11:54 AM, Jonathan Gordon wrote: On 6 October 2012 05:47, Richard Quirk wrote: A couple of patches to do with the default sleep duration setting, the first adds a "set_sleeptimer_duration" function that will be used in the second. http://gerrit.rockbox.org/327 http://gerr

Re: Fix for default sleep timer duration as a shortcut

2012-10-06 Thread Jonathan Gordon
On 6 October 2012 05:47, Richard Quirk wrote: > A couple of patches to do with the default sleep duration setting, the first > adds a "set_sleeptimer_duration" function that will be used in the second. > >http://gerrit.rockbox.org/327 >http://gerrit.rockbox.org/328 > > My use case is: I ha

Fix for default sleep timer duration as a shortcut

2012-10-05 Thread Richard Quirk
A couple of patches to do with the default sleep duration setting, the first adds a "set_sleeptimer_duration" function that will be used in the second. http://gerrit.rockbox.org/327 http://gerrit.rockbox.org/328 My use case is: I have restart sleep timer when switching on set to "yes",