On Thu, Apr 13, 2017 at 06:07:53PM +0200, Michael Bax wrote: > I primarily eliminated some unnecessary casts and sign protection, > substituted M_PI since it was used elsewhere, and switched to the > non-exception-raising nearbyint*(). The rest was mostly related > clean-up. M_PI on my systems is a double literal, so this will promote the other arguments to double, then another conversion to float before passing to nearbyintf, right? The original use of double instead of float was more about the definition of atan2, but now that we have atan2f the situation has changed. -- James Cameron http://quozl.netrek.org/