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

[Måns Rullgård]

Loren Merritt <lorenm at u.washington.edu> writes:

> On Tue, 15 Sep 2009, Mans Rullgard wrote:
>
>> +void ff_mdct_init_neon(MDCTContext *s);
> [...]
>> +if (ARCH_ARM) ff_mdct_init_arm(s);
>
> inconsistent.

Indeed, thanks.

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

We do that all over the place.

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