FS#13278 - Spurious error message after playing a song for the first time
After installing Rockbox for the first time and selecting a song, it will display “Error accessing playlist control file” and then proceed to play the song. There’s no actual ill effect to the message, and it goes away after the first time, but it is a minor annoyance.
The problem appears to have been introduced by 46085c897854 in the function playlist_modified():
bool playlist_modified(const struct playlist_info* playlist) { if (!playlist) { playlist = ¤t_playlist; if (!playlist->control_created && playlist_resume() == -1) return false; } ... }
The call to playlist_resume() will fail if the playlist control doesn’t exist or can’t be loaded correctly, and display a splash screen. In this case playlist_modified() will return false.
One easy fix is adding a ‘quiet mode’ parameter to playlist_resume() to avoid showing splash messages. I could submit a patch to do this, but the deeper problem is that playlist_resume() assumes the caller somehow knows there is actually something to resume. It might be worth fixing that instead of “just get rid of the annoying message”.
For example “Resume playback” on the root menu avoids spurious messages by checking global_status.resume_index != -1.
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