[FFmpeg-devel] [PATCH] ffmpeg.c -me option implemented as an AVOption
Michael Niedermayer
michaelni
Thu Jul 5 20:24:49 CEST 2007
Hi
On Thu, Jul 05, 2007 at 03:12:56PM +0200, Stefano Sabatini wrote:
> Hi all,
>
> I spent some time trying to understand how the AVOption system works,
> and now there is a patch which simplifies ffmpeg.c, replacing
> opt_motion_estimation with an AVOption setting (operated by
> opt_default).
>
> Named values for -me_method are now supported in a simpler and cleaner
> way through the AVOption system.
>
> 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
>
> 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)
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070705/907e4268/attachment.pgp>
More information about the ffmpeg-devel
mailing list