This is the mail archive of the
java@gcc.gnu.org
mailing list for the Java project.
Re: JNI patches
- To: mgd at swarm dot org (Marcus G. Daniels)
- Subject: Re: JNI patches
- From: Tom Tromey <tromey at redhat dot com>
- Date: 02 Apr 2001 17:11:00 -0600
- Cc: java at gcc dot gnu dot org
- References: <rfir8zdtlt4.fsf@cathcart.sysc.pdx.edu>
- Reply-To: tromey at redhat dot com
>>>>> "Marcus" == Marcus G Daniels <mgd@swarm.org> writes:
Marcus> 3. included in #2 is a patch to set env->locals to erase
Marcus> mention of dead frames. This was causing the
Marcus> _Jv_JNI_PopLocalFrame to crash.
I don't think this is the right patch. Instead I think we want to
update env->locals after the loop. Could you try this patch on your
test case? Don't forget to first back out your patch...
Tom
Index: jni.cc
===================================================================
RCS file: /cvs/gcc/gcc/libjava/jni.cc,v
retrieving revision 1.36.4.4
diff -u -r1.36.4.4 jni.cc
--- jni.cc 2001/04/02 22:51:54 1.36.4.4
+++ jni.cc 2001/04/02 23:01:32
@@ -335,6 +335,9 @@
rf = n;
}
+ // Update the local frame information.
+ env->locals = rf;
+
return result == NULL ? NULL : _Jv_JNI_NewLocalRef (env, result);
}