lib/android: `AssetManager::bitmap` only has to return a local reference
authorAlexis Laferrière <alexis.laf@xymus.net>
Mon, 16 Nov 2015 22:18:43 +0000 (17:18 -0500)
committerAlexis Laferrière <alexis.laf@xymus.net>
Mon, 16 Nov 2015 22:45:00 +0000 (17:45 -0500)
Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

lib/android/assets_and_resources.nit

index 39ee1c8..897938a 100644 (file)
@@ -173,10 +173,9 @@ class AssetManager
 
        # Return a bitmap from the assets
        private fun bitmap(name: String): NativeBitmap do
-               sys.jni_env.push_local_frame(1)
-               var return_value = new NativeBitmap.from_stream(native_assets_manager.open(name.to_java_string)).new_global_ref
-               sys.jni_env.pop_local_frame
-               return return_value
+               sys.jni_env.push_local_frame 2
+               var return_value = new NativeBitmap.from_stream(native_assets_manager.open(name.to_java_string))
+               return return_value.pop_from_local_frame
        end
 
        # Deallocate the global reference allocated by AssetManager