Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#7626 - fixing problem with "End of Song List" when viewing playlist without playback

Attached to Project: Rockbox
Opened by Xinlu Huang (polygonal) - Monday, 20 August 2007, 10:22 GMT+2
Last edited by Peter D'Hoye (petur) - Wednesday, 22 August 2007, 01:50 GMT+2
Task Type Patches
Category Playlists
Status Closed
Assigned To No-one
Player Type All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

Currently when you do contextmenu - playlist - view, the playlist will open but closes immediately if you are not playing anything (play/pause) with error/info message "End of Song List".

This turns out to be a bug in playlist_viewer.c, where a function that accepts an external filepath to open playlist checks for playing state, and if not playing/paused, exit and closes the playlist immediately.

I removed the part that checks for playing state since it's not actually needed except for one small later check.

This should fix  FS#7413 .

I tested it with my ipod 60GB, but no reason why it should not work on other targets as well. Comment below if you find anything wrong/no working as expected.
   fix_view_playlist_when_not_playing.patch (0.7 KiB) 
 apps/playlist_viewer.c |    9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

This task depends upon

Closed by  Peter D'Hoye (petur)
Wednesday, 22 August 2007, 01:50 GMT+2
Reason for closing:  Accepted
Comment by Dieter (dip) - Monday, 20 August 2007, 17:45 GMT+2
How do you access context menu - playlist - view if your are not in play or pause mode? On my iPod I can access this menu item only from the WPS screen which only available in play or pause mode. The context menu which can be accessed be pressing long SELECT on a file in the file browser or in the database view shows only the entries "playlist - insert" and "playlist - insert shuffled" when in stop modus but no "playlist - view" entry.
When I select the "playlist" menu from the main menu there is a menu entry "View Current Playlist" but this works.
Comment by Xinlu Huang (polygonal) - Monday, 20 August 2007, 17:48 GMT+2
Sorry, not clear enough. If you do context menu on a *playlist* in file browser, the problem occurs.
Comment by Xinlu Huang (polygonal) - Monday, 20 August 2007, 20:44 GMT+2
Fixed a small display bug. Now it should not display a now playing icon next to the last song in playlist if the playlist has finished playing.

   fix_view_playlist_when_not_playing1.patch (1.1 KiB) 
 apps/playlist_viewer.c |   11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

Comment by Peter D'Hoye (petur) - Tuesday, 21 August 2007, 01:33 GMT+2
hmmm I created a playlist, used the context menu on it and viewed the playlist - while no music was playing.
iriver H10 and h300.
Comment by Xinlu Huang (polygonal) - Tuesday, 21 August 2007, 01:39 GMT+2
From file browser? i.e. do context-menu on .m3u or .m3u8 files? And playback state is stopped, not paused? Does not work for me... and the code, if I understand correctly, check for global resume state, and if stopped ( = -1 ) exit the while loop (thus the viewer screen) immediately and does not wait for any button to be pressed.
Comment by Peter D'Hoye (petur) - Wednesday, 22 August 2007, 01:30 GMT+2
ok, so this only happens if playback can't be resumes. Reproduced and confirmed
Comment by Peter D'Hoye (petur) - Wednesday, 22 August 2007, 01:45 GMT+2
patch looks likes it fixes it.
I'll need your full name to accept the patch

Loading...