Details
-
Type:
Bug
-
Status:
Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 1.5.2, 1.5.3, 1.5.4
-
Fix Version/s: 1.5.5
-
Component/s: None
-
Labels:None
-
Patch Submitted:Yes
-
Number of attachments :
Description
I've sent this a couple times with no reply, so putting it here so it doesn't get lost.
The logic in ReflectionCache.getCachedClass is a bit suspect and has been the source of a couple bugs. Attached is a rearranged version that makes the logic clearer and slightly faster (it will never do more tests and will usually do fewer - like half as many).
In looking at the code (which is functionally identical to 1.5.3 which fixed GROOVY-2553) a question arises that indicates there is still a bug in that the case for primitive Byte.TYPE is missing. This may not be a bug of course since the test for 2553 seems to test it too, but then the question arises why Byte.TYPE is omitted since it appears in primitiveTypesMap as well as Number.byteValue() (I think it the former that is probably relevant here).
The patch is formatted for the 1_5_X branch to minimize whitespace differences rather than correct indentation.
Thinking more about what the question about the missing Byte.TYPE for isPrimitive meant lead me to a test case that demonstrates that there is still a bug after the fix for
GROOVY-2553.Attached is a test case (which I think is a good replacement for Groovy2553Bug.java).
GROOVY-2553. Attached is a test case (which I think is a good replacement for Groovy2553Bug.java).