On Fri, Mar 19, 2021 at 20:42:04 +1100, Daniel Playfair Cal wrote:
> -    const int visible = hypotf(uf, vf) <= 0.5f;
> +    const int visible = -0.5 < uf && uf < 0.5 && -0.5 < vf && vf < 0.5;

Please avoid double constants in float expressions, they propagate the
whole calculation to a double. I.e. use "0.5f" instead of "0.5".

(Also in one of your other patches.)

Cheers,
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