[FFmpeg-cvslog] r19955 - in trunk/libavcodec: dsputil.c dsputil.h

Diego Biurrun diego
Wed Sep 23 12:06:30 CEST 2009


On Wed, Sep 23, 2009 at 01:08:33AM +0100, M?ns Rullg?rd wrote:
> Michael Niedermayer <michaelni at gmx.at> writes:
> 
> >> I suggest moving ff_dot_productf() from celp_math.c to here instead of 
> >> duplicating it.
> >
> > no objection but i think dsp is getting somewhet bloated, it maybe should be
> > split one day ...
> > of course iam also not suggesting to overdo it and end up with a huge mess
> > of a hundread contexts ...
> 
> The thought of splitting it has occurred to me as well.  Perhaps
> separating audio and video things would be an option.  I can't think
> of any function that's used by both types of codecs.  The functions
> that are very specific, and only used by a single codec, should
> probably be moved away too.

Please split it.  This should also help with (re)compilation times,
which are horrible for dsputil.c.

Diego



More information about the ffmpeg-cvslog mailing list