On 12/08/2010 11:37 AM, Pekka Enberg wrote: > On Wed, Dec 8, 2010 at 1:13 PM, Andrew Haley <aph@xxxxxxxxxx> 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.
