[Android] [PATCH 3/3] jni: myVm can now be static

Thomas Guillem tom at gllm.fr
Tue Oct 14 11:35:39 CEST 2014


---
 vlc-android/jni/libvlcjni-medialist.c | 3 ---
 vlc-android/jni/libvlcjni.c           | 2 +-
 2 files changed, 1 insertion(+), 4 deletions(-)

diff --git a/vlc-android/jni/libvlcjni-medialist.c b/vlc-android/jni/libvlcjni-medialist.c
index f782f96..257fadc 100644
--- a/vlc-android/jni/libvlcjni-medialist.c
+++ b/vlc-android/jni/libvlcjni-medialist.c
@@ -28,9 +28,6 @@
 #define LOG_TAG "VLC/JNI/MediaList"
 #include "log.h"
 
-/** Unique Java VM instance, as defined in libvlcjni.c */
-extern JavaVM *myVm;
-
 struct stopped_monitor {
     pthread_mutex_t doneMutex;
     pthread_cond_t doneCondVar;
diff --git a/vlc-android/jni/libvlcjni.c b/vlc-android/jni/libvlcjni.c
index 9c28442..2b3a272 100644
--- a/vlc-android/jni/libvlcjni.c
+++ b/vlc-android/jni/libvlcjni.c
@@ -115,7 +115,7 @@ static void releaseMediaPlayer(JNIEnv *env, jobject thiz)
  * Note: It's okay to use a static variable for the VM pointer since there
  * can only be one instance of this shared library in a single VM
  */
-JavaVM *myVm;
+static JavaVM *myVm;
 
 static jobject eventHandlerInstance = NULL;
 
-- 
2.1.0



More information about the Android mailing list