[FFmpeg-devel] [PATCH] Move H.264 DSP functions from dsputil.c to h264dsp.c
Michael Niedermayer
michaelni
Sat Jul 28 13:34:41 CEST 2007
Hi
On Sat, Jul 28, 2007 at 11:49:10AM +0200, Panagiotis Issaris wrote:
> Hi,
>
> Michael Niedermayer wrote:
> > 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
> >
> > [...]
> >
>
> I'd like to start with renaming h264dsp.c to h264dspenc.c. So that would be:
> svn mv h264dsp.c h264dspenc.c and the attached patch. Okay to commit?
ok
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- 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/1d03cd76/attachment.pgp>
More information about the ffmpeg-devel
mailing list