[RFA/JVMTI] Fix a couple of thinkos
Keith Seitz
keiths@redhat.com
Wed Oct 11 22:04:00 GMT 2006
Hi,
My bad. These are probably pretty obvious to most people, but it looks
like I goofed two things in JVMTI-land. First, the JVMTI namespace I
introduced to jvmti-int.h for event processing must have all the members
declared extern (and subsequently defined in one place).
Second, we cannot simply lock the environment list in
_Jv_JVMTI_PostEvent. It would prevent another event from posting an
event because the lock would still be held by the suspended thread.
Keith
ChangeLog
* include/jvmti-int.h (JVMTI): Declare all members "extern".
* jvmti.cc (JVMTI): Define.
* jvmti.cc (_Jv_JVMTI_PostEvent): Don't lock the environment list.
(check_enabled_event): Likewise.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: jvmti-thinkos.patch
Type: text/x-patch
Size: 4192 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/java-patches/attachments/20061011/8e991088/attachment.bin>
More information about the Java-patches
mailing list