[FFmpeg-devel] [PATCH] ffmpeg.c -me option implemented as an AVOption

Michael Niedermayer michaelni
Fri Jul 6 10:36:39 CEST 2007


Hi

On Thu, Jul 05, 2007 at 10:43:19PM +0200, Stefano Sabatini wrote:
> 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.

patch ok if it works and passes regression tests

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- 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/20070706/939256b1/attachment.pgp>



More information about the ffmpeg-devel mailing list