dev builds
themes manual
device status forums
mailing lists
IRC bugs
dev guide

Rockbox mail archive

Subject: Re: Record on start patch

Re: Record on start patch

From: Jerry Van Baren <>
Date: Thu, 3 Feb 2005 10:44:50 -0500

Christi Alice Scarborough wrote:
> Hi,
> Could one of the developers spare a bit of time to look at the patch I
> submitted for adding a start in the record screen option, please? It
> seems a shame not to have it in the main code since it's a feature
> several people have expressed an interest in.
> Thanks,
> Christi

Hi Christi,

The patch looks good to me. I'll try it when I get home tonight.
Thanks for putting it together.

IIRC, in my personal hack, I added a call to recording_screen() in
app_main() before browse_root(). This starts you up in the recording
screen. If you press "OFF" to exit the recording screen, you will drop
into the normal main menu screen system. Using my hack, to get back to
the recording screen you have to traverse the normal menu rather than
cycling through the "rec screen" -> "rec menu" -> "main menu" sequence.
  I don't know which is better... since you actually have a usable
patch, I guess that makes your way better ;-). On the other hand, my
UNTESTED! proposed variant (below) makes the menu sequence work the same
as it currently does, with the exception that you start in the recording

Example of my hack combined with Christi's elegant patch (UNTESTED!):
void app_main(void)

     if (global_settings.rec_start && ! start_wps) {
          * Start in the recording screen. On exit, mimic the
          * normal progression of the recording menu and then the
          * main menu, at which point we are operating normally.

This would replace the part of your patch in apps/tree.c:
Index: apps/tree.c
RCS file: /cvsroot/rockbox/apps/tree.c,v
retrieving revision 1.280
diff -u -r1.280 tree.c
--- apps/tree.c 2 Jan 2005 12:08:34 -0000 1.280
+++ apps/tree.c 13 Jan 2005 15:43:13 -0000
_at__at_ -57,6 +57,7 _at__at_
  #include "talk.h"
  #include "filetypes.h"
  #include "misc.h"
+#include "recorder/recording.h"

  #include "widgets.h"
_at__at_ -951,9 +955,22 _at__at_


- if (*dirfilter < NUM_FILTER_MODES)
+ if (*dirfilter < NUM_FILTER_MODES) {

+#ifndef SIMULATOR
+ if (global_settings.rec_start && ! start_wps) {
+ /* We fake being in the menu structure by calling the
appropriate */
+ /* parent when we drop out of each screen */
+ recording_screen();
+ rec_menu();
+ main_menu();
+ }
+ }
      numentries = showdir(currdir, dirstart, dirfilter);
      if (numentries == -1)
          return false; /* currdir is not a directory */

Thanks again,

P.S. Christi: You are missing the "i" in your emailer's "display name"
setting, it is coming through as "Christ Alice Scarborough"
Received on 2005-02-03

Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy