On 12/08/2010 11:37 AM, Pekka Enberg wrote:
> On Wed, Dec 8, 2010 at 1:13 PM, Andrew Haley <[email protected]> wrote:
>>> http://developer.classpath.org/pipermail/classpath-patches/2010-November/006512.html
>>
>> What compatibility problem does this fix?
>
> That the result is not what we get with OpenJDK. JamVM, for example,
> (and I guess CACAO) has fixed this in their tree as has GCJ. The test
> case I used for this is ClassTest.testGetSimpleName() here:
>
> https://github.com/penberg/malva/blob/master/src/malva/java/lang/ClassTest.java
The patch is OK.
I'm not sure about this bit, but I can live with it:
@@ -299,24 +299,24 @@ final class VMClass
if (isAnonymousClass(klass))
return "";
if (isArray(klass))
- {
- return getComponentType(klass).getSimpleName() + "[]";
- }
+ return getComponentType(klass).getSimpleName() + "[]";
+
Andrew.
