[MPlayer-dev-eng] [PATCH] ve_xvid4.c: have each option flag have its counterpart

[Guillaume POIRIER]

Hi,
Le dimanche 31 octobre 2004 à 16:03 +0100, Guillaume POIRIER a écrit :
> Le jeudi 28 octobre 2004 à 19:49 +0300, Ivan Kalvachev a écrit :
> > On Sun, 24 Oct 2004 16:40:18 +0200
> > Guillaume POIRIER <guillaume.poirier at etudiant.univ-rennes1.fr> wrote:
> > 
> > > +	{"nochroma_opt", &xvidenc_chroma_opt, CONF_TYPE_FLAG, 1, 0, , NULL},
> > 
> > Don't like the ", ," otherwise it looks ok to commit.
> 
> Yes, that's why I sent an fixed patch yesterday! ;-)
> 
> I'll send the manpage update soon (tonight hopefully), and if Diego is
> ok with it, I'll commit both the front-end update and the docs update.

Here it is!

I plan to commit it, since I have Ivan blessing, in two days, or maybe
sooner if a core dev gives me the go-ahead.

Please note that I have noticed something that needs to get fixed about
the description of divx5's capability flag (its name changed in API 4,
and its usage has to be precised as this flag seems to be unnecessary
with recent releases of DivX).

Regards,
Guillaume
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ve_xvid4.c_counterpart_options+docs.patch
Type: text/x-patch
Size: 9401 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20041031/6c88a2f4/attachment.bin>