[FFmpeg-devel] [PATCH] Move H264 dsputil functions into their own struct
Tue Mar 16 01:49:39 CET 2010
Michael Niedermayer <michaelni at gmx.at> writes:
> On Tue, Mar 16, 2010 at 12:17:43AM +0000, M?ns Rullg?rd wrote:
>> Michael Niedermayer <michaelni at gmx.at> writes:
>> > On Sun, Mar 14, 2010 at 05:17:32PM +0000, Mans Rullgard wrote:
>> >> This moves the H264-specific functions from DSPContext to the new
>> >> H264DSPContext. The code is made conditional on CONFIG_H264DSP
>> >> which is set by the codecs requiring it.
>> >> The qpel and chroma MC functions are not moved as these are used by
>> >> non-h264 code.
>> > [...]
>> >> --- /dev/null
>> >> +++ b/libavcodec/h264dsp.c
>> > dont forget svn cp
>> So which files should be cp'd? Even the ones moving only a few lines
>> out of thousands?
> one moving 10 lines out of 1000 doesnt need to be cped
> also if the lines are stuff like just prototypes or comments then i care less
So in this specific case? Just dsputil.c -> h264dsp.c or the .h files
mans at mansr.com
More information about the ffmpeg-devel