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

Michael Niedermayer michaelni
Sat Jul 28 13:42:39 CEST 2007


Hi

On Sat, Jul 28, 2007 at 01:12:28PM +0200, Panagiotis Issaris wrote:
> Hi,
> 
> Panagiotis Issaris wrote:
> > [...]
> >   
> > Next, the attached patch moves the H.264 decoder DSP functions out of
> > dsputil into a separate file called h264dspdec.c.
> > Tested with ./configure --enable-gpl and ./configure --enable-gpl --disable-decoder=h264
> >
> >  Makefile     |    2
> >  dsputil.c    |  321 -----------------------------------------------------
> >  h264dspdec.c |  355 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> >  3 files changed, 360 insertions(+), 318 deletions(-)
> >
> >
> >   
> 
> Oh yes, for the header of h264dspdec.c I took the dsputil.c header and
> gave it a more descriptive and specific title.
> 
> One thing I wasn't sure about though was  the statement regarding
> the author of gmc & qpel as no qpel or gmc code (being Michael Niedermayer).
> 
> Michael: As I haven't moved qpel or gmc code to h264dspdec.c I did not
> copy that
> statement to h264dspdec.c. Was that the right choice?

the statement was just about mpeg4 asp stuff
and i think that none of the h264 code was touched by fabrice ...


> 
> And a question for an upcoming patch moving qpel H.264 code: Was that
> qpel statement
> about all qpel code, all but h264 qpel code or only the h264 qpel code?
> 
> Furthermore, the header of dsputil.c and so also h264dspdec.c has
> copyright notices for 2000-2004
> but nothing is mentioned after that period. Maybe it's better to extend
> it to 2007?

yes, maybe

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

Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070728/9e9e1fb2/attachment.pgp>



More information about the ffmpeg-devel mailing list