3 files changed, 8 insertions, 28 deletions

diff --git a/firmware/target/hosted/android/lcd-android.c b/firmware/target/hosted/android/lcd-android.c
index 66ddcd5e3e..f9d7917a5e 100644
--- a/firmware/target/hosted/android/lcd-android.c
+++ b/firmware/target/hosted/android/lcd-android.c
@@ -46,7 +46,6 @@ static bool display_on;
 void connect_with_java(JNIEnv* env, jobject fb_instance)
 {
     JNIEnv e = *env;
-
     /* Update RockboxFramebuffer_instance */
     if (!e->IsSameObject(env, RockboxFramebuffer_instance, fb_instance)) {
         if (RockboxFramebuffer_instance != NULL)
@@ -55,7 +54,7 @@ void connect_with_java(JNIEnv* env, jobject fb_instance)
         RockboxFramebuffer_instance = e->NewGlobalRef(env, fb_instance);
     }
 
-    static bool have_class = false;
+    static bool have_class;
     if (!have_class)
     {
         jclass fb_class = e->GetObjectClass(env, fb_instance);
@@ -77,13 +76,10 @@ void connect_with_java(JNIEnv* env, jobject fb_instance)
         java_lcd_init        = e->GetMethodID(env, fb_class,
                                              "java_lcd_init",
                                              "(IILjava/nio/ByteBuffer;)V");
-
-        jobject buffer       = e->NewDirectByteBuffer(env,
+                                               
+        native_buffer        = e->NewDirectByteBuffer(env,
                                                   lcd_framebuffer,
                                                   (jlong)sizeof(lcd_framebuffer));
-
-        native_buffer        = e->NewGlobalRef(env, buffer);
-
         have_class           = true;
     }
     /* we need to setup parts for the java object every time */
@@ -96,7 +92,6 @@ void lcd_deinit(void)
     JNIEnv *env_ptr = getJavaEnvironment();
 
     (*env_ptr)->DeleteGlobalRef(env_ptr, RockboxFramebuffer_instance);
-    (*env_ptr)->DeleteGlobalRef(env_ptr, native_buffer);
 }
 
 /*
diff --git a/firmware/target/hosted/android/pcm-android.c b/firmware/target/hosted/android/pcm-android.c
index 4b6df7f878..0f59f3f2ab 100644
--- a/firmware/target/hosted/android/pcm-android.c
+++ b/firmware/target/hosted/android/pcm-android.c
@@ -33,6 +33,7 @@ static char   *pcm_data_start;
 static jmethodID play_pause_method;
 static jmethodID stop_method;
 static jmethodID set_volume_method;
+static jclass    RockboxPCM_class;
 static jobject   RockboxPCM_instance;
 
 
@@ -157,11 +158,10 @@ void pcm_play_dma_init(void)
     JNIEnv *env_ptr = getJavaEnvironment();
     JNIEnv e = *env_ptr;
     /* get the class and its constructor */
-    jclass RockboxPCM_class = e->FindClass(env_ptr, "org/rockbox/RockboxPCM");
+    RockboxPCM_class = e->FindClass(env_ptr, "org/rockbox/RockboxPCM");
     jmethodID constructor = e->GetMethodID(env_ptr, RockboxPCM_class, "<init>", "()V");
     /* instance = new RockboxPCM() */
-    jobject PCM_instance = e->NewObject(env_ptr, RockboxPCM_class, constructor);
-    RockboxPCM_instance = e->NewGlobalRef(env_ptr, PCM_instance);
+    RockboxPCM_instance = e->NewObject(env_ptr, RockboxPCM_class, constructor);
     /* cache needed methods */
     play_pause_method = e->GetMethodID(env_ptr, RockboxPCM_class, "play_pause", "(Z)V");
     set_volume_method = e->GetMethodID(env_ptr, RockboxPCM_class, "set_volume", "(I)V");
@@ -170,13 +170,6 @@ void pcm_play_dma_init(void)
     pcm_play_get_more_callback((void*)&pcm_data_start, &pcm_data_size);
 }
 
-void pcm_deinit(void)
-{
-    JNIEnv *env_ptr = getJavaEnvironment();
-
-    (*env_ptr)->DeleteGlobalRef(env_ptr, RockboxPCM_instance);
-}
-
 void pcm_postinit(void)
 {
 }
diff --git a/firmware/target/hosted/android/system-android.c b/firmware/target/hosted/android/system-android.c
index f9f0605b49..c499631132 100644
--- a/firmware/target/hosted/android/system-android.c
+++ b/firmware/target/hosted/android/system-android.c
@@ -37,19 +37,13 @@ uintptr_t *stackend;
 extern int main(void);
 extern void telephony_init_device(void);
 extern void lcd_deinit(void);
-extern void pcm_deinit(void);
 
 void system_exception_wait(void) { }
 void system_reboot(void) { }
 
 void power_off(void)
 {
-    JNIEnv *env_ptr = getJavaEnvironment();
-
     lcd_deinit();
-    pcm_deinit();
-
-    (*env_ptr)->DeleteGlobalRef(env_ptr, RockboxService_class);
 }
 
 void system_init(void)
@@ -62,8 +56,6 @@ void system_init(void)
 JNIEXPORT void JNICALL
 Java_org_rockbox_RockboxService_main(JNIEnv *env, jobject this)
 {
-    jclass class = (*env)->GetObjectClass(env, this);
-
     /* hack!!! we can't have a valid stack pointer otherwise.
      * but we don't really need it anyway, thread.c only needs it
      * for overflow detection which doesn't apply for the main thread
@@ -73,8 +65,8 @@ Java_org_rockbox_RockboxService_main(JNIEnv *env, jobject this)
     stackbegin = stackend = (uintptr_t*) &stack;
 
     (*env)->GetJavaVM(env, &vm_ptr);
-    RockboxService_instance = (*env)->NewGlobalRef(env, this);
-    RockboxService_class = (*env)->NewGlobalRef(env, class);
+    RockboxService_instance = this;
+    RockboxService_class = (*env)->GetObjectClass(env, this);
 
     main();
 }