[MPlayer-dev-eng] [PATCH] manpage update?

[Arpi]

Hi,

> Changes:
>     - added mmst,cddb,user/pass to synopsis
ok
>     - added hints about .mp3 detection (-demuxer 17)
ok
>     - added rawaudio and dxr2 options (patch by albeu)
ok
>     - added lavdopts
ok
>     - tried to simplify/improve -npp, [as]lang/[as]id,
>       ao and more descriptions
please also add note that -alang/-slang works only for DVD playback, from
DVD device (already grabbed .vob file playback won't!) - so only with -dvd
title or dvd://title

(it gets language info from the device)

>     - added noise filter description (-vop)
>     - added vlelim,vcelim,vstrict,vdpart,vpsize description 
>       (lavcopts)

unfortunatelly most of teh diff is pure cosmetics, ading/removing spaces, so
it's really hard to follow the real changes :(

>     - added and removed some GUI keyboard controls

about keys:

+s               take screenshot
it's nonexistant, was removes some time ago from libvo

+m               mute sound
it was master/pcm switch, not mute, and also removed some time ago 

there was mention of -afm help, there is no such option
-afm expects integer number now
it will change after 0.90, when -afm will expect drver name instead of
number, and -afm help will work too

>     - tired to make a uniform manpage:
> 	uniform EXAMPLE:, NOTE:
> 	always 4 spaces before "sub-descriptions"
> 	<options> -> <option1:option2:...>
> 	always 2 spaces between first and second column
> 	always dot at the end of a option description
> 	.TP .B -> .TP\n.B

noticed :(

probably it rould be better to have 2 diffs, one with cosmetics and one with
orther chanegs, so verifying other changes would be MUCH simpler
this time i spent hours reading the diff, but next time if you decode to do
codmetics, try to do it in 2 passes

>     - many small fixes/typos/changes
> 
> Todo/questions:
>     - IMHO the lavcopts are too detailed, wouldn't a hint to the 
>       tech docs be better?
no
tech docs is not for users

>     - vfwopts, aq (lameopts) and gray (lavcopts)
there is gray for both lavcopts and lavdopts
vfwopts has no use, it is not working yet
aq is audio quality

>     - Wouldn't it be better to specify the fb device like this:
>       -vo fbdev[:device] instead of -vo fbdev -fb <device>?
yes
needs some volunteers to fix the code
i have no fbdev 

>     - looks ugly in small window
it should look ok in 80x25
smaller than 80 doesn't matter, but as manpage is mostly readed on console,
don't expect 132x50 or such

>     - IMHO it would be better to centralize all option 
>       declaration to cfg-*.h, now they're spread over many 
>       files, but donno.
it was albeu's "great" idea

>     - What about moving the -dump* options to mencoder?
is on TODO

>     - What about this mail (DVD subtitle ripping getting there):
>       http://www.mplayerhq.hu/pipermail/mplayer-dev-eng/2002-June/009004.html
dunno
btw there was a dvdsub ripping patch by Kimh, wa sit ever tested/commited?

> Hints:
>     - "vbug" is NO lavdoption, it's "bug"
possible


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu