[GROOVY-2001] NumberMath.getMath(Number) behaves differently from NumberMath.getMath(Number,Number) for Byte and Short

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0, 1.1-beta-2
• Fix Version/s: 1.5.1
• Component/s: groovy-jdk
• Environment: Groovy 1.0, Windows XP/Pro
• Assignee: Paul King
• Reporter: Steve Hanmann
• Created: 25/Jul/07 10:18
• Updated: 22/Dec/07 07:04
• Resolved: 15/Dec/07 04:03

Description

The NumberMath.getMath(Number) treats Byte and Short values as unexpected Number subclasses. Whereas, NumberMath.getMath(Number,Number) handles Byte and Short using IntegerMath.instance.

It would be desirable if NumberMath.getMath(Number) would behave like NumberMath.getMath(Number,Number), so that single Number argument methods were behaved consistently with the two Number argument methods.

For example, NumberMath.negate( Byte.valueOf("1") ) and NumberMath.negate( Short.valueOf("1") ) fail with an IllegalArgumentException.