[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