[MPlayer-dev-eng] [PATCH] Extra EDL command line argument
Reynaldo H. Verdejo Pinochet
reynaldo at opendot.cl
Fri Jun 25 03:24:24 CEST 2010
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
More information about the MPlayer-dev-eng
mailing list