[FFmpeg-devel] [PATCH 1/3] avcodec: add avdsp
Ronald S. Bultje
rsbultje at gmail.com
Sat Jul 26 15:58:16 CEST 2014
On Sat, Jul 26, 2014 at 9:34 AM, Michael Niedermayer <michaelni at gmx.at>
> This provides a public sustainable API/ABI for DSP functions.
> Only externally used dsp functions are included.
> The structure is extensible without ABI issues compared to the
> existing dsp contexts.
Please don't add new shithole-for-everything structs and interfaces, we
already have enough of these. Just call it "avsimpleidctcontext" or
"avsimplefdctcontext" or something, and put just idct or just fdct in it. I
don't mind the concept of externally visible structs at all, but this seems
designed as dsputil-with-an-av-prefix and that's scary.
Please don't reintroduce dsputil with most issues still there.
More information about the ffmpeg-devel