[MPlayer-DOCS] XviD encoding options man page

[Diego Biurrun]

Guillaume POIRIER writes:
> Diego Biurrun wrote:
> > Guillaume POIRIER writes:
> > 
> >>See the attached patch for better-documented options thanks to their 
> >>advices.
> > 
> > Comments below.
> Ok, I think I got it right now...

It's better, but it could still be improved.

> By the way, did you think about whether or not a section on "which 
> encoding options to use to make sure you can play back them fine on 
> hardware players" ?
> On the one hand, things evolving quite fast on the field of hifi dvd 
> players, this section might get outdated quite fast, one the other hand, 
> it's always better to warn beforehand... What do you think?

I had no time to ponder this issue yet, I will post an answer to your
other mail soon.

> -This option does not deinterlace video, it encodes it field-based
> -(default: off).
> +This option does not deinterlace video, it encodes it field-based.
> +(default: off)

There is no need to move the "(default: off)" out of the sentence,
same below.

I would just drop the "(default: off)" entirely.  It's clear that all
those options are off by default, else there would be options to turn
them off, not on.

>  .B reduced
> -enable encoding reduced resolution frames (default: off)
> +enable encoding reduced resolution frames
> +(default: off)

This is a complete sentence that should be capitalized and end in a
period.

Otherwise the patch is fine, please update the things I mentioned and
I will commit.
Thanks

Diego