Patch: FYI: Add missing JNICALL attributes to jvmti.cc

Mohan Embar gnustuff@thisiscool.com
Mon Jul 24 00:04:00 GMT 2006


Hi People,

I checked in the following patch under the "obvious" rule.
It's needed to unbreak the MinGW build after this one:

http://gcc.gnu.org/ml/java-patches/2006-q3/msg00191.html

-- Mohan
http://www.thisiscool.com/
http://www.animalsong.org/

2006-07-23  Mohan Embar  <gnustuff@thisiscool.com>

	* jvmti.cc (_Jv_JVMTI_SuspendThread): Add missing JNICALL
	attribute.
	(_Jv_JVMTI_ResumeThread): Likewise.
	(_Jv_JVMTI_DisposeEnvironment): Likewise.

Index: jvmti.cc
===================================================================
--- jvmti.cc	(revision 115684)
+++ jvmti.cc	(working copy)
@@ -29,7 +29,7 @@
 #define THREAD_CHECK_IS_ALIVE(thread)				\
   if (!thread->isAlive ()) return JVMTI_ERROR_THREAD_NOT_ALIVE;
 
-static jvmtiError
+static jvmtiError JNICALL
 _Jv_JVMTI_SuspendThread (MAYBE_UNUSED jvmtiEnv *env, jthread thread)
 {
   using namespace java::lang;
@@ -45,7 +45,7 @@
   return JVMTI_ERROR_NONE;
 }
 
-static jvmtiError
+static jvmtiError JNICALL
 _Jv_JVMTI_ResumeThread (MAYBE_UNUSED jvmtiEnv *env, jthread thread)
 {
   using namespace java::lang;
@@ -64,7 +64,7 @@
 #define RESERVED NULL
 #define UNIMPLEMENTED NULL
 
-static jvmtiError
+static jvmtiError JNICALL
 _Jv_JVMTI_DisposeEnvironment (jvmtiEnv *env)
 {
   // All we need to do is free memory allocated by _Jv_GetJVMTIEnv






More information about the Java-patches mailing list