[MPlayer-dev-eng] [PATCH] Extra EDL command line argument

Vlad Seryakov vseryakov at gmail.com
Fri Jun 25 03:36:31 CEST 2010


i have no problem with -edl-backward-delay but regarding doc text it needs someone, english is not my native language 

On Jun 24, 2010, at 9:24 PM, Reynaldo H. Verdejo Pinochet wrote:

> Hi Vlad
> 
> On Tue, Jun 22, 2010 at 10:26:42PM -0400, Vlad Seryakov wrote:
>> [...] 
>> 
>> ===================================================================
>> --- DOCS/man/en/mplayer.1	(revision 31528)
>> +++ DOCS/man/en/mplayer.1	(working copy)
>> @@ -1460,6 +1460,18 @@
>> on how to use this.
>> .
>> .TP
>> +.B \-edl\-backward\-extra\-sec <number>
>> +When using EDL during playback and jumping backwards it is possible to
>> +end up in the middle of an EDL record, in this case mplayer will jump out of 
> 
> I would s/this/that/ but I'm not sure.
> 
>> +the record in the same directon, to the start position of this EDL record. 
> 
> s/the record/it/
> 
>> +This parameter adds number of seconds to add to this seek so jumping out of 
> 
> Using this parameter you can set the number of seconds to add to
> the backward seek so..
> (just to avoid the adds/add in the same sentence)
> 
>> +the record will result in position a little before the start position and 
>> +will give some time to a user to press another button for going farther or
> 
> position .. position, needs to be rephrased aswell. I would also
> s/give some time to a user/give you some time/
> 
>> Index: DOCS/man/en/mplayer.1
>> +to pause. Without this extra seconds the continued after the seek playback 
> 
> _these_ extra seconds. But I would also rephrase this as:
> Without this delay the playback would.. (its kind of obvious you are refering
> to playback _after_ the seek)
> 
>> +would enter the same EDL record again and jump to the end immediately.
> 
> s/enter/jump into/ but that's just a matter of taste I guess.
> 
>> [..]
>>      CONF_TYPE_PRINT, 0, 0, 0, NULL},
>>     {"edlout", &edl_output_filename,  CONF_TYPE_STRING, 0, 0, 0, NULL},
>> +    {"edl-backward-extra-sec", &edl_backward_extra_sec,  CONF_TYPE_INT, CONF_MIN, 0, 0, NULL},
> 
> I would replace this option  and the var itself by edl-backward-delay like
> I commented on your previous patch.
> 
> Now, we really need some more peer review here. My english is far from
> perfect.
> 
> Best regards
> 
> --
> Reynaldo
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng




More information about the MPlayer-dev-eng mailing list