Rockbox

Tasklist

FS#11056 - Alarm clock plugin

Attached to Project: Rockbox
Opened by Clément Pit--Claudel (CFP) - Wednesday, 24 February 2010, 17:58 GMT
Last edited by Rafaël Carré (funman) - Thursday, 25 February 2010, 14:45 GMT
Task Type Patches
Category Plugins
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

This patch adds an "alarmclock" plugin, which basically resumes playback at a given time. Think of it as a play timer, as opposed to the sleep timer.

I found it useful, provided that it is no problem for me to leave my player on overnight. Should work on most targets with an RTC (however the code has not been tested but on Gigabeat), and shouldn't break compilation anyway.

Does the same as  FS#6044 , but that one seemed pretty old (I should maybe search FS before coding patches though).

This is a very simple plugin, that would better be merged into the core, for targets which are not able to automatically wake up. I do not have, however, the required talent (or knowledge of the code) to do this.

CFP.

PS: While developing this plugin, I noticed that for a few seconds after booting the sim, the clock dosen't update.

(by the way, the reported version list in the tacker posting interface doesn't show v3.5)
This task depends upon

Closed by  Rafaël Carré (funman)
Thursday, 25 February 2010, 14:45 GMT
Reason for closing:  Accepted
Additional comments about closing:  r24912
Comment by Clément Pit--Claudel (CFP) - Wednesday, 24 February 2010, 18:09 GMT
Woops
Comment by Clément Pit--Claudel (CFP) - Wednesday, 24 February 2010, 18:10 GMT
Wrong exit status (and I can't click the right button, it seems ; sorry).
Comment by Clément Pit--Claudel (CFP) - Wednesday, 24 February 2010, 19:27 GMT
Update the diff to add the rockbox header, and a manual section (which does need proofreading though).

One bug remains (as discussed on IRC): when starting the plugin, if no playlist control file has ever been created, an error message is displayed (error accessing playlist control file). Since the audio resume code was actually taken from the playback menu in plugin lib, this bug happens in every plugin: for example, try to start codebuster on a clean sim build, and resume the audio playback from the playback menu.

I believe this doesn't need fixing, provided that the error message is very neat.
Comment by Clément Pit--Claudel (CFP) - Wednesday, 24 February 2010, 19:55 GMT
Update the patch as per funman's comments.
Comment by Clément Pit--Claudel (CFP) - Wednesday, 24 February 2010, 20:00 GMT
After discussion on IRC, the "bug" is actually the desired behavior :)
CFP.
Comment by Clément Pit--Claudel (CFP) - Wednesday, 24 February 2010, 20:02 GMT
Tested on most sims, seems ok.
CFP.

Loading...