[MPlayer-dev-eng] [PATCH] remove obsolete lavcopts mbqmin/max

[Michael Niedermayer]

Hi

On Friday 10 September 2004 16:43, Alex Beregszaszi wrote:
> Hi,
>
> > > The lavc options mbqmin and mbqmax have done nothing since Lagrange
> > > multipliers were introduced (2003-10-7). They were then replaced by
> > > vqmin and vqmax.
> > >
> > > --Loren Merritt
> >
> > are you sure about this? i think you're wrong. mbqmin and mbqmax are
> > for adaptive quantization, only enabled with qprd, and *_mask options.
> > afaik they still work. vqmin and vqmax are definitely NOT the same
> > thing!

mbqmin/max are indeed unused currently, thats because there is just 1 
quantizer (rc->lambda->mb_lambda->mb_qp, not 2 (rc->frame_qp->mb_qp) anymore

>
> I would say removing options is a bad way. Of course you are free to
> update the documentation to point out these issues.

yes, agree

-- 
Michael
level[i]= get_vlc(); i+=get_vlc();  (violates patent EP0266049)
median(mv[y-1][x], mv[y][x-1], mv[y+1][x+1]); (violates patent #5,905,535)
buf[i]= qp - buf[i-1];    (violates patent #?)
for more examples, see http://mplayerhq.hu/~michael/patent.html
stop it, see http://petition.eurolinux.org & http://petition.ffii.org/eubsa/en