[FFmpeg-devel] [PATCH] Move H.264 DSP functions from dsputil.c to h264dsp.c

Panagiotis Issaris takis.issaris
Sat Jul 28 20:24:49 CEST 2007


Michael Niedermayer wrote:
> Hi
>
> On Sat, Jul 28, 2007 at 11:49:10AM +0200, Panagiotis Issaris wrote:
>   
>> Hi,
>>
>> Michael Niedermayer wrote:
>>     
>>> Hi
>>>
>>> On Fri, Jul 27, 2007 at 06:04:18PM +0200, Panagiotis Issaris wrote:
>>> [...]
>>>   
>>>       
>>>>>>>  void ff_h264dsp_init(DSPContext* c, AVCodecContext *avctx)
>>>>>>>  {
>>>>>>> +#if defined(CONFIG_H264_ENCODER)
>>>>>>>      c->h264_dct = h264_dct_c;
>>>>>>> +#endif
>>>>>>>           
>>>>>>>               
>>>>>> Here you could use if (ENABLE_H264_ENCODER).
>>>>>>         
>>>>>>             
>>>>> Unfortunately, this won't work, as the H.264 encoder isn't in Subversion
>>>>> yet.
>>>>>
>>>>>
>>>>>       
>>>>>           
>>>>>> Except those remarks, the patch looks fine to me.
>>>>>>         
>>>>>>             
>>>>> Thanks!
>>>>>       
>>>>>           
>>>> Ouch, I just noticed that with this patch I had accidentally moved lots
>>>> of the H.264 decoder DSP code in a file with a different header,
>>>> resulting in incorrect "Copyright by ..." lines.
>>>>
>>>> Is it okay to just merge the "Copyright by ..." lines? Or would it be
>>>> better to split the h264dsp.c file right away in a h264dspenc.c and
>>>> h264dspdec.c as Aurelien suggested?
>>>>     
>>>>         
>>> IMHO split if possible
>>>
>>> [...]
>>>   
>>>       
>> I'd like to start with renaming h264dsp.c to h264dspenc.c. So that would be:
>> svn mv h264dsp.c h264dspenc.c and the attached patch. Okay to commit?
>>     
>
> ok
>
> [...]
>   

Applied.

With friendly regards,
Takis




More information about the ffmpeg-devel mailing list