[MPlayer-DOCS] CVS: main/DOCS/man/en mplayer.1,1.715,1.716

[Diego Biurrun]

Daniele Forghieri CVS writes:
> 
> Modified Files:
> 	mplayer.1 
> Log Message:
> All video driver suboptions documented, gif and tga examples

:-)))))

I have some nits to pick as usual, but I have already made all the
changes in my local tree, so don't bother fixing this up yourself,
just please answer the questions I have below.

> -.B sdl (SDL only)
> +.B sdl (SDL only, also see \-forcexv, \-noxv)

Should be

.B sdl (also see \-forcexv, \-noxv) (SDL only)

to be consistent with the rest of the man page.

> +.IPs <device>
> +Explicitly choose the fbdev device name to use (e.g.\& /dev/\:fb0) or the
> +name of the vidix subdevice if the device name start with vidix (e.g.\&

starts

> +vidixsis_vid to enable the sis driver).

Does this have a default?  It should be mentioned then.

>  .TP
>  .B vesa\ \ \ 
> @@ -2423,10 +2445,14 @@
>  Turns on DGA mode.
>  .IPs nodga
>  Turns off DGA mode.
> +.IPs neotv_pal
> +Activate the neomagic's TV out and set it to PAL norm.
> +.IPs neotv_ntsc
> +Activate the neomagic's TV out and set it to NTSC norm.

NeoMagic TV out

> +Use the VIDIX driver.
> +.IPs lvo:\ \ \ 

I assume the colon is a mistake, right?

> +Activate the Linux Video Overlay on top of vesa mode.

VESA

> +.IPs <subdevice>
> +Explicitly choose the Blinkenlights subdevice driver to use.
> +It's something like arcade:host=localhost:2323 or
> +hdl:file=name1,file=name2. 

This should be changed into a proper example.

> +.IPs <device>
> +Explicitly choose the matrox device name to use (default: /dev/\:mga_vid).

Matrox

> +.IPs <device>
> +Explicitly choose the matrox device name to use (default: /dev/\:mga_vid).

Matrox

>  .B 3dfx (Linux only)
>  3Dfx specific video output driver.
> -FIXME: What's the difference between 3dfx, tdfxfb and tdfx_vid?
> +This driver uses directly the 3dfx hardware on top of X11.

directly uses, 3Dfx

> +This driver use directly the tdfx_vid kernel module.

directly uses

> +Output video sub driver to use as overlay (x11, xv).

subdriver

> -.IPs device=<number>
> +.IPs 0\ 1\ 2\ 3\ 
>  Specifies the device number to use if you have more than one em8300 card.

Is this really limited to 0-3?  And shouldn't it be device=X?

> +.IPs <filename>
> +If you don't have a DVB card this video output driver writes a file in MPEG-PES
> +format on the file you specify (default: ./grab.mpg).

This should go to the main description.

> +.IPs PAL\ \ \ 

Does this have to be uppercase?  Otherwise it should be written in
lowercase, to be consistent.

> +Activate pal video norm.

PAL

> +.IPs SECAM
> +Activate secam video norm.

SECAM

> +.IPs NTSC\ \ \ 
> +Activate ntsc video norm.

NTSC

> +mplayer video.nut \-vo gif89a:15.0:test.gif

:-)

Diego