[MPlayer-DOCS] CVS: main/DOCS/man/en mplayer.1,1.604,1.605

[The Wanderer]

Diego Biurrun CVS wrote:

> -Another great feature of MPlayer is the wide range of supported output
> -drivers.
> +MPlayer supports a wide range of video and audio output drivers.
>  It works with X11, Xv, DGA, OpenGL, SVGAlib, fbdev, AAlib, libcaca, DirectFB,
>  Quartz, but you can also use GGI, SDL (and all their drivers), VESA (on every
>  VESA compatible card, even without X11), some low level card-specific drivers

"VESA-compatible", "low-level"

> @@ -188,14 +187,12 @@
>  .
>  .SH "KEYBOARD CONTROL"
>  MPlayer has a fully configurable, command driven, control layer

"command-driven", I think - and the following comma is inappropriate 
regardless.

> @@ -362,8 +359,8 @@
>  Specify configuration file to be parsed after the default ones.
>  .TP
>  .B \-quiet \ \ 
> -With this option the status line (i.e. A:   0.7 V:   0.6 A-V:  0.068 ...)
> -will not be displayed.
> +Make MPlayer output less verbose, in particular the status line
> +(i.e. A:   0.7 V:   0.6 A-V:  0.068 ...) will not be displayed.

Suggest "MPlayer's output"; also, possibly use "verbose; in particular, 
the".

Additionally, I'd either use "Makes" or change the latter half to "in 
particular, prevents the status line ( ... ) from being displayed.", to 
avoid a tense inconsistency. That might be more bothering with than it 
deserves, though.

>  Particularly useful on slow terminals or broken ones that do not properly
>  handle carriage return (i.e. \\r).

"that" > "which"; this is a nitpick, but is technically correct.

> @@ -543,11 +538,13 @@
>  mechanism.
>  .TP
>  .B \-playlist <file>
> -Play files according to a playlist (1 file per row or Winamp or ASX format).
> +Play files according to a playlist file (1 file per line, Winamp or ASX format).

These do not seem equivalent to me. I think that it was specifying that 
you can use any of these three formats; at any rate, unless I'm much 
mistaken, ASX is not "one file per line" except in the sense that you 
aren't going to have *more* than one per line. I don't know the Winamp 
playlist format, but I somehow doubt it's simply an unformatted list 
with one file(name) per line.

> -.B \-slave \ \ 
> +.B \-slave (also see \-input)
>  This option switches on slave mode.
> -This is intended for use of MPlayer as a backend to other programs.
> -Instead of intercepting keyboard events, MPlayer will read simplistic command
> -lines from its stdin.
> +The aim is being able to use MPlayer as a backend for other programs.
> +Instead of intercepting keyboard events, MPlayer will read commands from stdin.
> +See DOCS/tech/slave.txt and \-input cmdlist for a list of slave commands.

Suggest "being able" > "be able".

>  .TP
>  .B \-softsleep
> -Uses high quality software timers.
> +Use high quality software timers instead of RTC.

"high-quality". Also, suggest "the RTC".

-- 
       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.