[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