[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