Rockbox

  • Status Closed
  • Percent Complete
    0%
  • Task Type Patches
  • Category
  • Assigned To No-one
  • Operating System
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by dmilamj - 2003-03-04
Last edited by zagor - 2003-06-04

FS#1014 - Recording Timer

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.

Closed by  zagor
2003-06-04 13:48
Reason for closing:  Accepted

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.

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
Tracker.

Here’s a link for anybody else who wants to view the patch:
http://www.geog.ubc.ca/~kujala/cvs_20030521_rectimer.diff

Project Manager

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.

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

Same URL i previously posted points to clean rectimer patch:
http://www.geog.ubc.ca/~kujala/cvs_20030521_rectimer.diff

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.

-vincent, http://www.geog.ubc.ca/~kujala

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing