[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