[MPlayer-DOCS] r24349 - in trunk: DOCS/man/en/mplayer.1 libmpcodecs/ve_lavc.c

[Guillaume POIRIER]

Hi,

On 9/8/07, Guillaume POIRIER <poirierg at gmail.com> wrote:
> Hi,
>
> On 9/6/07, reimar <subversion at mplayerhq.hu> wrote:
> > Author: reimar
> > Date: Thu Sep  6 10:57:50 2007
> > New Revision: 24349
> >
> > Log:
> > Support lowdelay flag
> >
> > Modified: trunk/DOCS/man/en/mplayer.1
> > ==============================================================================
> > --- trunk/DOCS/man/en/mplayer.1 (original)
> > +++ trunk/DOCS/man/en/mplayer.1 Thu Sep  6 10:57:50 2007
> > @@ -8861,6 +8861,10 @@ this setting can be a good alternative t
> >  quality at fixed bitrate) at a lesser cost than with VHQ (default: on).
> >  .
> >  .TP
> > +.B (no)lowdelay
> > +Sets the low delay flag for MPEG-1/2 (disables B-frames, default: off).
> > +.
> > +.TP
> >  .B (no)cartoon
> >  Activate this if your encoded sequence is an anime/\:cartoon.
> >  It modifies some Xvid internal thresholds so Xvid takes better decisions on
>
>
> This looks missplaced. You added a lavc option to xvid section.
> Attached patch fixes this.
> Additionnaly, it removes the "default: disabled" we the convention is
> that all options are disabled by default unless marked differently.

Applied.

I'm supposed to be a maintainer of these sections anyhow .... ;-)

Thanks for comitting both doc+code BTW :-)))

Guillaume
-- 
A soldier will fight long and hard for a bit of colored ribbon.
 -- Napoleon Bonaparte