[FFmpeg-devel] [PATCH] ffmpeg.c -me option implemented as an AVOption
Stefano Sabatini
stefano.sabatini-lala
Thu Jul 5 22:43:19 CEST 2007
On date Thursday 2007-07-05 20:24:49 +0200, Michael Niedermayer encoded:
> Hi
>
> On Thu, Jul 05, 2007 at 03:12:56PM +0200, Stefano Sabatini wrote:
[...]
> > One problem is that the patch is *not backward compatile*, since it
> > changes the OptionDef option "me" with the predefined "me_method"
> > AVOption defined in libavcodec/utils.c.
> >
> > If you think this is a problem I can modify the patch in order to
> > support the old option name with something as:
> >
> > {"me", "set motion estimation method (deprecated, use me_method insted)",
> > OFFSET(me_method), FF_OPT_TYPE_INT, ME_EPZS, INT_MIN, INT_MAX, V|E, "me_method"},
> >
> > in libavcodec/utils.c:options.
>
> yes, please add this but under #if LIBAVCODEC_VERSION_INT < ... so that it
> automatically disapears with the next major version bump
Done, hope it is OK.
> > BTW: I noted that there are many OptionDef options in ffmpeg.c which
> > can be implemented as AVOptions ones, resulting in simpler code. Is it
> > OK to change them in this way too?
>
> yes (note dont forget the regression tests)
OK (patches incoming...) ;-).
But before I want to manage this (obviously into another thread):
sds at santefisi:~/tmp$ ffmpeg -b bogus
[...]
ffmpeg: unrecognized option '-b'
Cheers.
--
Stefano Sabatini
Linux user number 337176 (see http://counter.li.org)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: me-method-as-avoption-01.patch
Type: text/x-diff
Size: 5763 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070705/aa7be37b/attachment.patch>
More information about the ffmpeg-devel
mailing list