This is the mail archive of the
java-patches@gcc.gnu.org
mailing list for the Java project.
Patch: FYI: Add missing JNICALL attributes to jvmti.cc
- From: Mohan Embar <gnustuff at thisiscool dot com>
- To: Java Patch List <java-patches at gcc dot gnu dot org>
- Cc: tromey at redhat dot com, Keith Seitz <keiths at redhat dot com>
- Date: Sun, 23 Jul 2006 19:04:02 -0500
- Subject: Patch: FYI: Add missing JNICALL attributes to jvmti.cc
- Reply-to: gnustuff at thisiscool dot com
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