[MPlayer-dev-eng] [PATCHES] Edit Decision Lists

Olivier Rolland billl at users.sf.net
Sun Jun 22 20:35:24 CEST 2008


Hi,

Here are two patches dealing with EDL support in mencoder.

The first one is related to this bug
http://bugzilla.mplayerhq.hu/show_bug.cgi?id=4: it gets rid of
potential timestamp resets by using demuxer->stream_pts (if defined)
instead of demuxer->video->sh->pts. Ultimately,
demuxer->video->sh->pts (and the like) are replaced by calls to
demuxer_get_current_time (demuxer). It's a little bit different with
the audio stream because of the calc_a_pts function, that's why I'm
using static variables to store previous values to detect pts resets.
All in all, with this patch, the functionnal behaviour is exactely the
same with movies without timestamp resets.

I've made a few tests with DVDs, AVIs, OGMs and MKVs with and without
-hr-edl-seek and didn't have any regression.

The second patch is less functional. It simply cleans out the EDL code
in mencoder by removing unneeded parameters and tests, replacing a
goto with a while and moving the EDL code from the main loop to its
own function.

I guess something similar can be done for mplayer.

Regards,

Olivier Rolland
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mencoder-edl-seek.patch
Type: text/x-patch
Size: 9308 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080622/ba61b1b6/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mencoder-edl-clean.patch
Type: text/x-patch
Size: 6862 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080622/ba61b1b6/attachment-0001.bin>


More information about the MPlayer-dev-eng mailing list