Rockbox

  • Status Closed
  • Percent Complete
    100%
  • 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
Attached to Project: Rockbox
Opened by mikeholden - 2006-11-16
Last edited by learman - 2007-08-22

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.

Closed by  learman
2007-08-22 16:35
Reason for closing:  Fixed
Additional comments about closing:  

Fix committed, with some minor changes.

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?

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing