[cp-patches] Problem in native/jni/classpath/jcl.c

[Ian Rogers]

There's a bug in the Classpath CVS head that's crashing the Jikes RVM
(thanks to Mark for helping me to find it). It's in jcl.c where we
delete a global ref which is actually only a local ref. The following
fixes it.

Thanks,

Ian Rogers

Index: native/jni/classpath/jcl.c
===================================================================
RCS file: /sources/classpath/classpath/native/jni/classpath/jcl.c,v
retrieving revision 1.25
diff -u -r1.25 jcl.c
--- native/jni/classpath/jcl.c  17 Sep 2006 19:13:01 -0000      1.25
+++ native/jni/classpath/jcl.c  27 Sep 2006 09:34:32 -0000
@@ -75,7 +75,6 @@
     {
       jclass tmp = rawDataClass;
       rawDataClass = (*env)->NewGlobalRef (env, rawDataClass);
-      (*env)->DeleteGlobalRef (env, tmp);
     }

   if (rawDataClass != NULL)
@@ -90,7 +89,6 @@
     {
       jclass tmp = rawDataClass;
       rawDataClass = (*env)->NewGlobalRef (env, rawDataClass);
-      (*env)->DeleteGlobalRef (env, tmp);
     }

   if (rawDataClass != NULL)