- Status Closed
- Percent Complete
- Task Type Patches
- Category
-
Assigned To
zagor - Operating System
- Severity Medium
- Priority Low
- Reported Version
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#769 - Autobookmarks (multiresume) and Bookmarking, with diff
This time, with the diff.
Autobookmarks (multiresume) and Bookmarking
If you have any questions/suggestions, contact me at
rockbox_dev@samuraipanda.com. 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
bookmark.
Bookmarking.
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
will
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
load.
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.
FIX NOTE:
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.
FIX NOTE:
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.
bookmarks.diff
(69.8 KiB)
bookmarks.diff
(102.1 KiB)
bookmarks.diff
(62.9 KiB)
bookmarks.diff
(66.8 KiB)
bookmarks.diff
(61.4 KiB)
bookmarks.diff
(63.4 KiB)
bookmarks.diff
(63.5 KiB)
bookmarks.diff
(72.2 KiB)
bookmarks.diff
(67.7 KiB)
bookmarks.diff
(59.9 KiB)
bookmarks.diff
(69.2 KiB)
2004-01-14 08:04
Reason for closing: Accepted
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
Logged In: YES
user_id=1157
Committed.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
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!!
I’ll be posting a link to a compiled version next week…Ben
Thank you very, very much!!!
I have posted a compiled version
at:http://www.samuraipanda.com/bookmarks.zip.
- Ben
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?
Thanks
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
http://www.samuraipanda.com/bookmarks.zip. 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.
Ok, thank you very much, Myself, as well as many other
people commend your efforts, and appreciate immensely
what you are doing.
THANK YOU!!
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
-Cameron
clidestri@yahoo.com
A new version has been posted. A compiled version can be
found at:
http://www.samuraipanda.com/bookmarks-20030207.zip - Ben
A new version has been posted. A compiled version can be
found at:
http://www.samuraipanda.com/bookmarks-20030220.zip - Ben
An unpdated compiled version can be found at:
www.samuraipanda.com/bookmarks-20030227.zip
Thanjks so much for updating regularily! All of us using
Rockbox apreciate you greatly!!
Thanks Again
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.
Lot’s of changes have occured in the bookmarking code.
An unpdated compiled version and a list of changes can be
found at:
www.samuraipanda.com/bookmarks-20030316.zip
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!!
Is there a new version of the compiled bookmark enabled
firmware available? I’ve tried all the URLs below
I did 2003-06-26 builds which include both my text viewer
patch and Ben’s bookmarking patch:
http://www.adfreeinternet.com/~garyd/rockbox/
Could you please update this patch. It dosen’t apply without
some work now.
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.
Dynamic playlists are not supported. They are by nature
dynamic (changable) and therefor bookmarking them would
prove futile.
will somebody please FINISH this???!!
I also listen to a LOT of spoken-word audio, and would love to
see this modification included in Rockbox.
All major issues with this patch are now resolved and we’re
currently just ironing out a few final things before it goes
in.
Stay tuned.
Committed.