On Thu, May 16, 2019 at 12:51:46 +0200, Lynne wrote:
> > +#define STEPSIZE    0.3010299957
>
> Just hardcode this in the powf call.

With an appended 'f'.

> > +    const float scale = 1.0f;

> const float scale = 1.0 / 32768;

1.0f, but it doesn't matter here, as the preprocessor reduces it to
float (alledgedly).

> > +        s->deviation_inverse[i] = 1.f / s->standard_deviation[i];
>
> Nit: 1.0 instead of 1.f

No, 1.f or 1.0f. Otherwise, it promotes the division to a double
precision operation before casting back to the left hand side float.

(It only matters on hardware which requires double precision software
emulation, but if explicitly using float, make sure to stick to float.)

Moritz
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to