[MPlayer-dev-eng] [RFC] EDL seeking and timing

Vlad Seryakov vseryakov at gmail.com
Wed Sep 1 21:57:17 CEST 2010


On Sep 1, 2010, at 1:17 PM, Reynaldo H. Verdejo Pinochet wrote:

> Hi Vlad
> 
> On Tue, Aug 31, 2010 at 03:58:23PM -0400, Vlad Seryakov wrote:
>> This patch adds new command line option-edl-adjust-pts, when specified all positions in the edl file will be added with start_pts for playback, and if used with -edlout, positions will be saved minus start_pts. In summary, this options converts positions in all EDL files to be relative starting from 0. 
>> Without this, everything works as it was before.
> 
> OK
> 
>> [...]
>> Index: cfg-mplayer.h
>> ===================================================================
>> --- cfg-mplayer.h	(revision 32037)
>> +++ cfg-mplayer.h	(working copy)
>> @@ -103,6 +103,8 @@
>>      CONF_TYPE_PRINT, 0, 0, 0, NULL},
>>     {"edlout", &edl_output_filename,  CONF_TYPE_STRING, 0, 0, 0, NULL},
>>     {"edl-backward-delay", &edl_backward_delay,  CONF_TYPE_INT, CONF_MIN, 0, 0, NULL},
>> +    {"edl-adjust-pts", &edl_adjust_pts,  CONF_TYPE_FLAG, 0, 0, 1, NULL},
> 
> Wouldn't using CONF_TYPE_FLOAT with a 0 default serve the same purpose
> and add the possibility of using a diferent start without making the
> added code more complex at all?
> 

You mean storing actual start pts in this conf variable? Still need a flag to enable this feature so i decided for now keep it simple
I can make it double, for this code it does not matter.
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: edladjust.txt
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100901/8e53215e/attachment.txt>
-------------- next part --------------



More information about the MPlayer-dev-eng mailing list