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

Vlad Seryakov vseryakov at gmail.com
Mon Jun 21 22:03:41 CEST 2010


Thanks,  i actually like the process :-))

> 
> Got it. It sounds like a command line option with a sane default
> (2?) would be OK. We can leave that for latter though.
> 
>> +// Number of seconds to add to the seek when jumping out
>> +// of EDL scene in backward direction. This is needed to
>> +// have some time after the seek to decide what to do next
>> +// (next seek, pause,...), otherwise after the seek it will 
>> +// enter the same scene again and skip forward immediately
>> +float edl_backward_extra_sec = 2;
> 
> Maybe edl_backward_delay? your call. Anyway, could someone confirm
> this is correct Doxy? Diego? I don't recall.
> 

I actually initially thought about _delay but decided to keep it in similar way with rel_seek_sec and the rest

>> [...]
>> +    // This indicates that we need to reset next EDL record according 
>> +    // to new PTS due to seek or other condition
>> +    if (edl_needs_reset) {
>> +        edl_needs_reset = 0;
>> +
> 
> Drop the extra line.

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


More information about the MPlayer-dev-eng mailing list