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

Benoit Fouet benoit.fouet
Fri Jul 6 13:48:06 CEST 2007


Stefano Sabatini wrote:
> On date Friday 2007-07-06 10:36:39 +0200, Michael Niedermayer encoded:
>   
>> 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
>>     
>
> Regression test succeeded with my working source tree copy.
>
> Fixed a typo signaled by Marc.
>   

Applied

-- 
Ben
Purple Labs S.A.
www.purplelabs.com




More information about the ffmpeg-devel mailing list