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

Michael Niedermayer michaelni
Sat Jul 28 02:13:07 CEST 2007


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

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

In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- 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/fc31362c/attachment.pgp>



More information about the ffmpeg-devel mailing list