[FFmpeg-devel] [PATCH] Move H264 dsputil functions into their own struct

Michael Niedermayer michaelni
Tue Mar 16 01:44:03 CET 2010


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


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100316/fb9cacc5/attachment.pgp>



More information about the ffmpeg-devel mailing list