[Ffmpeg-devel] [PATCH] Remove the [mb]lm(in|ax) OptionDef

[Diego Biurrun]

On Thu, Sep 28, 2006 at 09:47:15AM +0200, V?ctor Paesa wrote:
> 
> > On Wed, Sep 27, 2006 at 11:38:06PM +0200, Panagiotis Issaris wrote:
> >>
> >> The attached patch removes the "mblmax" OptionDef. The equally named
> >> AVOption takes over its function. A unit change is involved, but as
> >> AVOption parameters are now parsed with ff_eval2(), you can use the
> >> QP2LAMBDA constant to ease the change.
> >>
> >> Regression tests succeed.
> >>
> >>  ffmpeg.c           |   13 -------------
> >>  libavcodec/utils.c |    2 +-
> >>  2 files changed, 1 insertion(+), 14 deletions(-)
> >
> > the [mb]lm(in|ax) change looks ok
> >
> 
> I would suggest to explain the unit change in the documentation,
> something like this:

Thanks, applied.

Diego