FS#769 - Autobookmarks (multiresume) and Bookmarking, with diff

Attached to Project: Rockbox
Opened by Benjamin Metzler (benjamintm) - Thursday, 16 January 2003, 22:47 GMT
Last edited by Björn Stenberg (zagor) - Wednesday, 14 January 2004, 08:04 GMT
Task Type Patches
Status Closed
Assigned To Björn Stenberg (zagor)
Operating System
Severity Medium
Priority High
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 0%
Votes 0
Private No


This time, with the diff.

Autobookmarks (multiresume) and Bookmarking
If you have any questions/suggestions, contact me at The diff file was
generated off of rockbox-daily-20030116.

Autobookmarking (Multi File Resume)
This patch creates the ability to autobookmark a file
when stopping that file with the OFF button. The code
takes the existing resume information and stores that to
a file based off of the file name (foo.mp3 creates a file
called foo.mp3.resume). Each time an MP3/M3U is
played, the code will check for the the existence of the
above file and, if found, will prompt the user to load the

This patch has the preliminary bookmarking capability.
It can make a bookmark file with multiple bookmarks,
but will only load the first bookmark in the file. This
be addressed in the next release of the patch (once I
figure out how to do menus!).

Know Issues/TBDs
1. For some reason, bookmarks will occasionally fail to

2. Need to create a menu when selecting a bookmark
file what will let the user choose which bookmark within
that file they want to play. Currently this loads the first
bookmark in the bmark file.

3. Manual bookmarks work, but only for MP3s. For
some reason, if attempting to bookmark a playlist, the
bookmark is created correctly, but the RockBox starts
playing at the last resume point.

4. Develop own better play capability.
I have duplicated the code for play_resume()
without the ask_resume() calls. Again, this is not the
best way to do this, but my knowledge of how to
actually play is lacking, so I haven't developed a more
robust method

5. Need to use the language files.
I've tried to use #define as much as possible, so it
should be trivial to use the LANG files.

6. Bookmark files are created, but they don't show up
until the next time the directory is loaded.

7. Need to come up with an icon for bookmark and
resume files

8. Need to store the autobookmark settings. They will
be restored to the defaults each time the RockBox is
turned on.
This task depends upon

Closed by  Björn Stenberg (zagor)
Wednesday, 14 January 2004, 08:04 GMT
Reason for closing:  Accepted
Additional comments about closing:  Logged In: YES

Comment by Anonymous Submitter - Saturday, 18 January 2003, 21:13 GMT

WOW, seems like an awesome patch!! benjamintm, could
you please compile it for us non-programmers, so we could
also use this wonderful feature?

Thanks alot!!
Comment by Benjamin Metzler (benjamintm) - Sunday, 19 January 2003, 23:51 GMT

I'll be posting a link to a compiled version next week...Ben
Comment by Anonymous Submitter - Monday, 20 January 2003, 03:35 GMT

Thank you very, very much!!!
Comment by Benjamin Metzler (benjamintm) - Tuesday, 21 January 2003, 17:43 GMT

I have posted a compiled version

- Ben
Comment by Anonymous Submitter - Sunday, 26 January 2003, 17:58 GMT

It may be just my imagination, but it seems that whenevre i
use this firmware, i hear the hard drive spinning alot mor
often, as aposed to the daily builds. It seems that every 4 or
5 seconds, the hard drive spins up, where as in the daily
builds, it seems like it takes 1 - 2minutes. Is this a bug, or is
it supposed to do that? also, will it ruin my Recorder?

Comment by Benjamin Metzler (benjamintm) - Sunday, 26 January 2003, 20:13 GMT

There was a bug in the first couple of releases where there
was more access to the hard disk then necessary. This was
fixed in the later releases. Download the latest version from I've been
updating a lot recently so there are a bunch of
changes/additions/fixes. Again, this is patch and may have
bugs. They shouldn't ruin your Recoder, but anything is
possible. If you are not comfortable with this, then stick with
the 1.4 release as that is supposed to be stable.
Comment by Anonymous Submitter - Monday, 27 January 2003, 04:51 GMT

Ok, thank you very much, Myself, as well as many other
people commend your efforts, and appreciate immensely
what you are doing.

Comment by Anonymous Submitter - Monday, 27 January 2003, 17:50 GMT

This is absolutely a MUST HAVE feature! It's working great for
me so far (using a AJBR15).
Who decides what features go into the next release? I want to
cast my vote for this!
I listen to Old Time Radio shows in the car to and from work,
and music during the day. I always had to hunt my place on
the show when I got back in the car... not any more!
Thanks, Benjamin

Comment by Benjamin Metzler (benjamintm) - Friday, 07 February 2003, 22:15 GMT

A new version has been posted. A compiled version can be
found at:
- Ben
Comment by Benjamin Metzler (benjamintm) - Thursday, 20 February 2003, 19:16 GMT

A new version has been posted. A compiled version can be
found at:
- Ben
Comment by Benjamin Metzler (benjamintm) - Friday, 28 February 2003, 05:33 GMT

An unpdated compiled version can be found at:
Comment by Anonymous Submitter - Tuesday, 04 March 2003, 05:00 GMT

Thanjks so much for updating regularily! All of us using
Rockbox apreciate you greatly!!

Thanks Again
Comment by Daniel Stenberg (bagder) - Tuesday, 04 March 2003, 16:11 GMT

A few comments on the 0227 version:

1. don't memset() strings without reason all over
2. the comment about the bookmark format is wrong, as your
code uses "," as separator while the comment says "*". I
also think you should make the separator a proper define.

Another more minor issue is that the code rules say "less
than 80 columns" and you don't. In my editor it makes your
long lines wrap weirdly.

Also, can you elaborate on what you mean with your "TBD"
item 1 in the bookmark.c file?

Taken all into account, I think this patch is about to get
ready for inclusion.
Comment by Benjamin Metzler (benjamintm) - Monday, 17 March 2003, 07:05 GMT

Lot's of changes have occured in the bookmarking code.
An unpdated compiled version and a list of changes can be
found at:
Comment by Anonymous Submitter - Tuesday, 25 March 2003, 15:24 GMT

I would like to simply add a comment regarding the
importance of Rockbox firmware and Benjamin Metzler's
patch for bookmarks. I have many digital audiobooks, some
of which are 18 hrs long per track. It is extremely important
to have a resume feature to return to the last read portion as
well as bookmarks. Bookmarks are especially important to
get to specific sections. For example, many of the
audiobooks have 15-20 stories spanning over 15 hrs, again,
all on one track! With bookmarks, I can easily navigate
through the books to get to my place in the audio file. I
sincerely appreciate the efforts to allow these features. For
audiobook lovers and old time radio show lovers, your
firmware has made the archos jukebox the ultimate dream
player. Thanks!!
Comment by Anonymous Submitter - Monday, 23 June 2003, 18:26 GMT

Is there a new version of the compiled bookmark enabled
firmware available? I've tried all the URLs below
Comment by Garrett Derner (garrettderner) - Friday, 27 June 2003, 15:31 GMT

I did 2003-06-26 builds which include both my text viewer
patch and Ben's bookmarking patch:
Comment by Lee Pilgrim (diddystar5) - Monday, 04 August 2003, 21:03 GMT

Could you please update this patch. It dosen't apply without
some work now.
Comment by Anonymous Submitter - Wednesday, 05 November 2003, 08:50 GMT

bug report :
It cannot bookmark on recursively, dynamically generated
playlists on archos Recorder. For instance I use On+Play on a
folder, choose insert. Then it's playing music, bookmarks
cannot be used.
Comment by Benjamin Metzler (benjamintm) - Wednesday, 05 November 2003, 20:35 GMT

Dynamic playlists are not supported. They are by nature
dynamic (changable) and therefor bookmarking them would
prove futile.
Comment by Anonymous Submitter - Monday, 22 December 2003, 15:49 GMT

will somebody please FINISH this???!!
Comment by Blue Chip (bluechip) - Monday, 12 January 2004, 07:46 GMT

I also listen to a LOT of spoken-word audio, and would love to
see this modification included in Rockbox.
Comment by Björn Stenberg (zagor) - Monday, 12 January 2004, 08:34 GMT

All major issues with this patch are now resolved and we're
currently just ironing out a few final things before it goes

Stay tuned.
Comment by Björn Stenberg (zagor) - Wednesday, 14 January 2004, 08:04 GMT