[MPlayer-dev-eng] [PATCH] EDL extensions, resubmit

Vlad Seryakov vseryakov at gmail.com
Sat Jun 19 19:03:05 CEST 2010


Hi Reynaldo

I've made i think all changes you've asked, it was my first patch submit and i missed the guidelines, sorry.
The reason i removed edl_seek function because to properly handle in-the-midle-of-the-scene situation i will have to duplicate code to handle reset, and i just moved edl_reset logic with extensions inside edl_update and introduced a flag that triggers reset. Also because reset may trigger another seek it needs to be inside edl_update, i thought about putting reset logic in separate function but then i need return code handling if i triggered a seek and code become more complicated than it needs to be.

To summarize the patch (from original submit):

I am working on a system that allows tagging video files with scenes and media box will use it and skip scenes depending on user options.
While researching i discovered mplayer's support for EDL records and after playing with it i found out that this is perfect player for our system. 
But to make sure the skipping content is hidden especially if we get into sensitive ara making scenes for kids for example, i tried to improve EDL support a little.
The result is the following:

- All original functionality is preserved
- When making a seek, jumping back/forward and resulting position ends up inside any scene, it gets kicked out immediately, depending on the direction, to the end or beginning of the scene
- Just after the seek and before any valid PTS is received, video flipping is disabled but only when EDL records are used, without EDL everything runs as before

I hope these changes will be helpful for everybody who is interested in EDL functionality.


-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: edl_patch.txt
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100619/a9a468e9/attachment.txt>
-------------- next part --------------




More information about the MPlayer-dev-eng mailing list