[FFmpeg-devel] [PATCH 1/2] Separate per-arch MDCT init from FFT init

Loren Merritt lorenm
Wed Sep 16 01:01:29 CEST 2009


On Tue, 15 Sep 2009, Mans Rullgard wrote:

> +void ff_mdct_init_neon(MDCTContext *s);
[...]
> +if (ARCH_ARM) ff_mdct_init_arm(s);

inconsistent.

Also, you're depending on the compiler to optimize away if(0), or you'll 
get a link error. It just so happens that current gcc does, even with 
-O0, but I wouldn't bet on it everywhere.

--Loren Merritt



More information about the ffmpeg-devel mailing list