[MPlayer-DOCS] CVS: main/DOCS/man/en mplayer.1,1.722,1.723

[The Wanderer]

Not as thorough about commenting on non-changed lines as usual, but a
few points nonetheless.


Diego Biurrun CVS wrote:

> +Enables the specified chain of postprocessing subfilters.
> +Subfilters must be separated by '/' and can be disabled by prepending
> +a '\-', appending a ':' followed by 'a', 'c' or 'y' determines its
> +scope (default: c):

This latter sentence seems poorly phrased, from my end. When I see
"prepending a '\-', appending a ':'", I expect it to be followed by ",or
<verb>ing a <noun>" or something along those lines. The segment
beginning with "appending" appears to be a separate idea, and should
probably be treated as such.

Also, I think it's formally a little ambiguous as to what is the
antecedent for "its".

>  .PD 0
>  .RSs
>  .IPs a
> -Automatically switches the filter off if the CPU is too slow.
> +Switch the subfilter off if the CPU is too slow.

I think this is less clear than the previous version... with the new
form, I wouldn't be sure whether it was "will react to the CPU being too
slow by turning off this filter" or "if your CPU is too slow, add this
to switch the filter off". The latter is plainly ridiculous in the
context (since its only use would be to change a default, if the CPU
were fast enough for some files but not for others), but it takes me a
little more thought to realize that than I'd really like.

I might suggest "Switch the subfilter off when printing the "your CPU is
too slow" message", except that a) I don't know if that message is
always printed (e.g. with -really-quiet) and b) there's been discussion
of vastly reducing that message or even removing it entirely, so making
part of the documentation assume it's there seems a little undesirable.
(Plus, while more clear, it's also less pretty than before.)

> -.IPs "\-vf pp=hb:y/\:vb:a \-autoq 6"
> -Deblock horizontal only luminance and switch vertical deblocking on or
> +.IPs "\-vf pp=hb:y/vb:a"
> +Horizontal deblocking on luminance only and switch vertical deblocking on or
>  off automatically depending on available CPU time.

Suggest a comma after "only".

> -Detects and drops first of interlaced frames in video stream.
> +Detects and drops first interlaced frame in a video stream.

I don't know if this is quite correct; after all, it takes two things
(frames) to be interlaced, because you have to interlace one with the
other. I think the plural might be correct. (Then again, I also just
woke up.)

-- 
       The Wanderer

Warning: Simply because I argue an issue does not mean I agree with any
side of it.

A government exists to serve its citizens, not to control them.