- Status Closed
- Percent Complete
- Task Type Bugs
- Category Music playback
-
Assigned To
learman - Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#6341 - Playlist completion creates bookmark
Using today’s daily build (061116).
I play a m3u playlist. When normal playback completes, a bookmark is created (I have bookmarking on stop set to auto), which is a couple of seconds before the end of the last song. If I play the playlist again, playback resumes a couple of seconds from the end of the last song, and then of course completes playback almost immediately.
I can work round this by either deleting the bookmark, setting it to ask, or not loading the bookmark on playback, but it’s very annoying having to almost disable a very useful feature just to get “normal” use of the box.
basically a bookmark should only be created when the STOP button is pressed, not when normal playback completes.
2007-08-22 16:35
Reason for closing: Fixed
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
Fix committed, with some minor
changes.
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
This is not specific to H300, someone with permission might want to
change that.
I agree the current behavior is undesirable.
Currently it’ll try to autobookmark on the main exit path out of
gui_wps_show(). Aside from the STOP action, this is basically when
audio_status() becomes 0. That might not make sense.
Other places that autobookmark: SYS_POWEROFF in gui_wps_show(), and
ACTION_TREE_STOP in dirbrowse().
Note that this means if you’ve gone to the file browser and left the
music playing, and then power off, no bookmark is saved. I’ve actually
been bitten by this once or twice.
When should it autobookmark? On STOP surely, in the WPS or file
browser. On power off too probably, at least if auto-resume is
disabled. What about other cases where playback is interrupted: entering
the recording screen or the radio, changing to another track from the
file browser or playlist viewer, starting a new playlist?
Anyway here’s a small patch that does the following:
-Have WPS autobookmark only on STOP.
-Take the SYS_POWEROFF case out of gui_wps_show(), and have it
autobookmark inside clean_shutdown() instead, so it’s done from any
context.
-Have bookmark.c’s system_check() prevent trying to bookmark if playback
isn’t currently in progress.
Forgot to mention when I logged this that this is only a recent problem. It used to work correctly some time recently, probably no more than a couple of weeks ago.
Unfortunately I don’t have a dev environment, so am unable to test this patch.
is this patch still required?