[MPlayer-dev-eng] [PATCH] Documentation of XviD options in man page

[Diego Biurrun]

Guillaume POIRIER writes:
> Diego Biurrun wrote:
> > Please start new sentences on a new line and break those overly long
> > lines, also for consistency write (default: off) instead of
> > (default=off).
> Ok, I made the changes. Please tell me if it looks good now!

It's better, but please start new sentences on a new line, nroff needs
this.

> > For extra point you could send this to MPlayer-DOCS,
> > but I will pick it up on either mailing list.
> Well, I'm not subscribed to MPlayer-DOCS, so I post it here for now. If 
> you happend to ask me to send more patches like this one, I'll 
> eventually send them there.

Apparently you seem to care about the XviD encoding options.  It would
be great if you could review and update all of them.  At the moment
they are in a sad state of neglect and very low on my TODO-list.  So
if you could tackle this area it would be very much appreciated.

Diego

> -Save per-frame statistics in xvid.dbg (default=off).
> +Save per-frame statistics in xvid.dbg (default: off).

Hmm, I suppose this is written to the current directory, so this
should be "./xvid.dbg".

> -enable quarter-pixel motion estimation (default=off)
> +MPEG4 uses a half pixel precision for its motion search by default. The 
> +standard proposes a mode where encoders are allowed to use quarter pixel 
> +precision.
> +This option usually results in sharper image. But it has great impact on 
> +bitrate and sometimes the higher use of bitrate will prevent it from giving a 
> +better image quality at a fixed bitrate. The better is to test w and w/o it and 
> +choose after this test, if it's worth activting it. (default: off)

As said above, please start new sentences on a new line.  Extra points
for avoiding trailing whitespace.

Diego