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

[Måns Rullgård]

Diego Biurrun <diego at biurrun.de> writes:

> 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.

The amount of code would still be the same, but I'll see what I can
do.  When I have the time...

-- 
M?ns Rullg?rd
mans at mansr.com