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