[MPlayer-dev-eng] [PATCH] Documenting -noaspect

[D Richard Felker III]

On Fri, Sep 20, 2002 at 08:35:09PM +0200, Arpi wrote:
> Hi,
> 
> > --- DOCS/mplayer.1	10 Sep 2002 21:59:52 -0000	1.232
> > +++ DOCS/mplayer.1	17 Sep 2002 19:00:53 -0000
> > @@ -1167,6 +1167,9 @@
> >  .PD 1
> >  .
> >  .TP
> > +.B \-noaspect
> > +Disable aspect compensation.
> > +.TP
> >  .B \-aspect <ratio>
> >  Override aspect ratio of movies. It's autodetected on MPEG files, but
> > can't be
> >  autodetected on most AVI files.
> 
> hmm
> teh default policy of manpage is not documenting the -no... options, it's
> written at teh beginning that -no is general, disables the option...

Hmm, for yes/no flags this is obvious, but since -aspect takes a
number, it might not be so obvious that -noaspect exists. Perhaps it
should at least be mentioned in the description of -aspect if not on
its own. IMO, though, including -no* options in the manpage is good
when they're not simply the negation of another option, but do
something extra, like in this case.

Rich