On 5/6/2019 6:26 PM, Michael Niedermayer wrote: > also i agree with paul, the API looks good > implementation not really reviewed I don't like the avfft_* namespace. It's unlike other modules we have. Since av_fft_* is taken by the lavc implementation and can't be used, maybe this one could be avutil_fft_* instead?