[FFmpeg-devel] [PATCH] avcodec/magicyuv: add SIMD for median of 10bits

Ronald S. Bultje rsbultje at gmail.com
Sat Dec 24 01:00:01 EET 2016


On Fri, Dec 23, 2016 at 12:32 PM, Paul B Mahol <onemda at gmail.com> wrote:

> diff --git a/libavcodec/lossless_videodsp.h b/libavcodec/lossless_
> videodsp.h

> @@ -32,6 +32,7 @@ typedef struct LLVidDSPContext {

> +    void (*add_magy_median_pred_int16)(uint16_t *dst, const uint16_t
> *top, const uint16_t *diff, unsigned mask, int w, int *left, int *left_top);

That seems wrong. Why would you add a magicuv-specific function to
losslessdsp-context which is intended for functions shared between many
(not just one) lossless codecs? You probably want a new dsp for magicyuv

I know this is tedious, but we're very specifically trying to prevent
dsputil from ever happening again.


More information about the ffmpeg-devel mailing list