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

Panagiotis Issaris takis.issaris
Sat Jul 28 11:56:57 CEST 2007


Hi,

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?
>
>  Makefile  |    2 -
>  h264dsp.c |   81
> --------------------------------------------------------------
>  2 files changed, 1 insertion(+), 82 deletions(-)
>
>
>   
The previous patch would then be followed by this one, which takes care of
some renames after the previous pure rename patch.

 dsputil.c    |    4 ++--
 h264dspenc.c |    4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)


With friendly regards,
Takis

-------------- next part --------------
A non-text attachment was scrubbed...
Name: pi-20070728T115428-ffmpeg-some_leftovers_after_rename_h264dspenc.diff
Type: text/x-patch
Size: 1361 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070728/51cf137a/attachment.bin>



More information about the ffmpeg-devel mailing list