[MATH-482] FastMath.max(50.0f, -50.0f) => -50.0f; should be +50.0f

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.2, 3.0
• Fix Version/s: 2.2, 3.0
• Assignee: Sebb
• Reporter: Sebb
• Created: 17/Jan/11 19:52
• Updated: 23/Mar/11 20:34
• Resolved: 17/Jan/11 20:01

Description

FastMath.max(50.0f, -50.0f) => -50.0f; should be +50.0f.

This is because the wrong variable is returned.

The bug was not detected by the test case "testMinMaxFloat()" because that has a bug too - it tests doubles, not floats.