On Thu, Nov 09, 2017 at 03:22:21AM +0100, Michael Niedermayer wrote: > On Wed, Nov 08, 2017 at 11:41:16PM +0100, Aurelien Jacobs wrote: > > On Wed, Nov 08, 2017 at 06:26:03PM +0100, Michael Niedermayer wrote: > > > On Wed, Nov 08, 2017 at 02:06:09PM +0100, Aurelien Jacobs wrote: > > > [...] > > > > +} > > > > + > > > > +/* > > > > + * Compute the convolution of the signal with the coefficients, and > > > > reduce > > > > + * to 24 bits by applying the specified right shifting. > > > > + */ > > > > +av_always_inline > > > > +static int32_t aptx_qmf_convolution(FilterSignal *signal, > > > > + const int32_t coeffs[FILTER_TAPS], > > > > + int shift) > > > > +{ > > > > + int32_t *sig = &signal->buffer[signal->pos]; > > > > + int64_t e = 0; > > > > + > > > > > > > + for (int i = 0; i < FILTER_TAPS; i++) > > > > > > "for (int" is something we avoided as some comilers didnt like it, > > > iam not sure if this is still true but there are none in the codebase > > > > If you insist on this I will of course change it, but hey, we require > > me personally, not really, i used "for (int" style as long as > i can remember outside ffmpeg but i also would like to keep supportng > all platforms ...
OK, got it. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel