FS#8607 - MPEG video playlist

Attached to Project: Rockbox
Opened by Andree Boeger (Candy) - Thursday, 14 February 2008, 08:50 GMT
Last edited by Teruaki Kawashima (teru) - Friday, 26 November 2010, 12:49 GMT
Task Type Patches
Category Video
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


If play mode = file, mpegplayer plays a video and stops at the end of the video, returning to the file browser (as it is the state now). In case of play mode = directory mpegplayer assumes a playlist which is identical to the current directory (lexically order of the files). This is the same behaviour, as if rockbox plays an audio file selected from the file browser. The playback stops when the last file in the directory has been played.

Having this playlist, you can easyly skip to the next or the previous video file by pressing the FF or RW button shortly.

It makes sense to switch the resume option to "Play from beginning", if you want to watch one video after another, but it's not neccessary.
This task depends upon

Closed by  Teruaki Kawashima (teru)
Friday, 26 November 2010, 12:49 GMT
Reason for closing:  Accepted
Additional comments about closing:  Committed in r28667
Comment by Robert Kukla (roolku) - Monday, 18 February 2008, 16:24 GMT
I find this quite useful - as long as there are only few files in the directory and the skip is infrequent I think the approach is sensible (as opposed to maintaining a playlist like in the jpg viewer). The skip order is the order files are stored in the directory which is not neccessary alphabetically for FAT.
Comment by Michael Sevakis (MikeS) - Saturday, 17 May 2008, 19:57 GMT
I think implementing as an engine control layer would be the proper approach - not in the UI layer. Additionally, a full stream_init should _not_ be needed between files but just a stream_close/stream_open. It is designed that way in anticipation of this sort of feature.
Comment by Michael Sevakis (MikeS) - Saturday, 17 May 2008, 20:20 GMT
I should mention that I tried to apply this patch to SVN r17557 without success so I could have misread some aspects of the .diff.
Comment by Taylore (trailblaze) - Sunday, 21 December 2008, 15:55 GMT
hmm... This patch looks really kool, however i got lots'o' hunk errors also... Can someone with patch experience fix this please.?
Comment by Jacob Brooks (jac0b) - Monday, 12 January 2009, 21:16 GMT
Comment by Taylore (trailblaze) - Friday, 16 January 2009, 21:56 GMT
lol. Thank you for resyncing it. It worked very nicely!, while it did.. I guess it it broken again,,, since i got a Hunk #10 fail at 1892 in apps/plugins/mpegplayer/mpegplayer.c (on build 19778)... Hmphh.. would you mind re-resyncing it please? thank you in advance...
Comment by Jacob Brooks (jac0b) - Saturday, 17 January 2009, 16:42 GMT
Comment by Jacob Brooks (jac0b) - Saturday, 18 April 2009, 16:25 GMT
Comment by Taylore (trailblaze) - Sunday, 21 June 2009, 19:55 GMT
hmm.. It seems to be broken again.. One hunk error while patching.. May someone please sync this again please?
Comment by Jacob Brooks (jac0b) - Thursday, 23 July 2009, 23:40 GMT
I tried to figure it out but they have rewritten the api so it is pretty well mangled. This is where the change happened;r2=21306;pathrev=21306
Comment by Teruaki Kawashima (teru) - Friday, 28 August 2009, 12:08 GMT
resync and change.
* short press of button for rewinding starts current video from the beginning if postition is not near the beginning regardless the resume option.
Comment by Teruaki Kawashima (teru) - Wednesday, 13 January 2010, 13:29 GMT
Update patch.
* Don't change video by pressing left or right button if in single mode.
* Add item to change video to menu.
* don't do stream_init and stream_exit when change video. don't know if it is done correctly.
Comment by Teruaki Kawashima (teru) - Friday, 18 June 2010, 14:47 GMT
synced to r26919.
Comment by Teruaki Kawashima (teru) - Monday, 01 November 2010, 15:11 GMT
update patch.