> I'm pretty sure none of us would want to meet it in code review, save for a few niche edge cases.

It does no I/O and has no action-at-a-distance. The float-long-float casts are the worst part.

If a coworker brought it to me with a performance justification and some unit tests I'd be pretty happy to pass it.