[MPlayer-DOCS] [PATCH] XviD: divx5bvop description update + closed_gop

[Dominik 'Rathann' Mierzejewski]

Hi,

On Saturday, 06 November 2004 at 13:14, Guillaume POIRIER wrote:
> Hi,
> Le vendredi 05 novembre 2004 ? 23:31 +0100, Dominik 'Rathann'
> Mierzejewski a écrit :
[...]
> Thanks a lot for all your feedbacks.

You're welcome. :)

> Here's a fixed patch, that precises

There's no such verb as "to precise". I think the word you meant to use is
"refines". :P

Anyway, some further nitpicking below.

[...]
> diff -Nur -x CVS -x '.*' main/DOCS/man/en/mplayer.1 main_modif/DOCS/man/en/mplayer.1
> --- main/DOCS/man/en/mplayer.1	2004-11-02 18:22:23.000000000 +0100
> +++ main_modif/DOCS/man/en/mplayer.1	2004-11-06 12:48:59.000000000 +0100
[...]
> @@ -6557,7 +6556,7 @@
>  by any MPEG4 decoder besides libavcodec or XviD.
>  .br
>  .I WARNING:
> -This option is deprecated in XviD-1.0.x.
> +As of XviD-1.0.x, this option is unavailable.

Small nit: "no longer available" sounds nicer, but that's subjective I guess
and though I do know English quite well, I'm not a native speaker.

>  .
>  .TP
>  .B keyframe_boost=<0\-1000>
[...]
> @@ -6597,7 +6604,12 @@
>  .
>  .TP
>  .B (no)packed
> -Create a bitstream which can be decoded delay-free.
> +This option is meant to solve frame-order issues when encoding to
> +container formats like AVI that can't cope with out-of-order frames.
> +In practical, most decoders (both software and hardware) are able to deal

Like Diego said: "In practice, most..." Actually, he said "practise", which
is American spelling for this noun.

> +with frame-order themselves, and may get confused when this option is
> +turned on, so you can safely leave if off, unless you really know what
> +you're doing.
>  .br
>  .I WARNING:
>  This will generate an illegal bitstream, and will not be
[...]
> @@ -6635,6 +6643,14 @@
>  (default: 0)
>  .
>  .TP
> +.B (no)closed_gop
> +This option tells XviD to close every GOP (Group Of Pictures bounded
> +by two I-frames), which makes GOPs independent from each other.
> +This just implies that the last frame of the GOP is eiter a P-frame or a
> +N-frame but not a B-frame.
> +It is usually a good idea to turn on this option (default: on).

...turn this option on.

> +.
> +.TP
>  .B frame_drop_ratio=<0\-100> (max_bframes=0 only)
>  This setting allows the creation of variable framerate video streams.
>  The value of the setting specifies a threshold under which, if the

Regards,

R.

-- 
MPlayer RPMs maintainer: http://greysector.rangers.eu.org/mplayer/
"I am Grey. I stand between the candle and the star. We are Grey.
 We stand between the darkness ... and the light."
        -- Delenn in Grey Council in Babylon 5:"Babylon Squared"