FS#1014 - Recording Timer

Attached to Project: Rockbox
Opened by Dave Jones (dmilamj) - Tuesday, 04 March 2003, 17:21 GMT
Last edited by Björn Stenberg (zagor) - Wednesday, 04 June 2003, 13:48 GMT
Task Type Patches
Status Closed
Assigned To No-one
Operating System
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Votes 0
Private No


A patch that adds simple record timer functionality. The
record timer (if set to active) stops a recording at a given
interval and then starts recording again with a new file.

This is useful for long term recordings. The break
between recordings is only the time required to stop and
restart the recording, on the order of 2-4 seconds.

This adds two options to the "recording settings"
menu, "Record Timer Active?" and "Record Timer
Interval." These settings are stored in the hard disk
settings area. Currently there are six interval settings (5,
10, 15, and 30 minutes, 1, 2, and 4 hours) but this can
be expanded to 8 with the current setup.

This task depends upon

Closed by  Björn Stenberg (zagor)
Wednesday, 04 June 2003, 13:48 GMT
Reason for closing:  Accepted
Comment by Dave Jones (dmilamj) - Wednesday, 05 March 2003, 00:52 GMT

Also, the recorded file size is not displayed on the recording
screen if the record timer is active. It is replaced with "Rec
Timer: [Time}" so that it is obvious that the rec timer is active
and the currently set interval.

I've attached a .diff file based on the March 4th CVS code.
Comment by Vincent Kujala (vkujala) - Friday, 23 May 2003, 02:34 GMT

I have updated the patch for CVS-030521 with a few extra
unrelated changes: 1100mAh battery, lower DEFAULT_CONTRAST
status_draw(true) in recorder.c (few battery,volume,clock while
recording). Not quite sure how to submit this patch with

Here's a link for anybody else who wants to view the patch:
Comment by Linus Nielsen Feltzing (linusnielsen) - Friday, 23 May 2003, 22:42 GMT

By adding those unrelated changes, you just made it much
harder for us to apply and accept this patch in the main CVS
tree. Please remove them and resubmit the patch with only
the recording timer stuff. Thanks.
Comment by Vincent Kujala (vkujala) - Saturday, 24 May 2003, 17:06 GMT

Ok, I've separated my ugly diff into separate patches.

Same URL i previously posted points to clean rectimer patch:

My home page has links to above and other patches including
what i think is a bug (742131) in settings.c config_block[0xae].

This rectimer patch should co-exist with caption_backlight
(shuffled rectimer bits over one in config_block[0xae]).

Thanks Dave for the original rectimer patch.