From c2d4cce429baa56b5a0962a854ece6163270995f Mon Sep 17 00:00:00 2001
From: dhewg
Date: Wed, 2 Mar 2011 22:26:15 +0100
Subject: ANDROID: On pause, put all threads in a coma

Since not every engine respects pauseEngine(), or they're in a state
where it simply gets ignored, put all threads in a group coma. Without
this, code still kept looping and wasting cpu cycles, while the user
might want to do use her/his droid for something else.
---
 backends/platform/android/jni.cpp | 29 ++++++++++++++++++++++++-----
 1 file changed, 24 insertions(+), 5 deletions(-)

(limited to 'backends/platform/android/jni.cpp')

diff --git a/backends/platform/android/jni.cpp b/backends/platform/android/jni.cpp
index 1e28b065b4..9ecd9577e7 100644
--- a/backends/platform/android/jni.cpp
+++ b/backends/platform/android/jni.cpp
@@ -48,6 +48,9 @@ jobject JNI::_jobj_egl_surface = 0;
 Common::Archive *JNI::_asset_archive = 0;
 OSystem_Android *JNI::_system = 0;
 
+bool JNI::pause = false;
+sem_t JNI::pause_sem = { 0 };
+
 int JNI::surface_changeid = 0;
 int JNI::egl_surface_width = 0;
 int JNI::egl_surface_height = 0;
@@ -417,6 +420,10 @@ void JNI::create(JNIEnv *env, jobject self, jobject asset_manager,
 				jobject at, jint audio_sample_rate, jint audio_buffer_size) {
 	assert(!_system);
 
+	pause = false;
+	// initial value of zero!
+	sem_init(&pause_sem, 0, 0);
+
 	_asset_archive = new AndroidAssetArchive(asset_manager);
 	assert(_asset_archive);
 
@@ -477,6 +484,8 @@ void JNI::destroy(JNIEnv *env, jobject self) {
 	g_system = 0;
 	_system = 0;
 
+	sem_destroy(&pause_sem);
+
 	// see above
 	//JNI::getEnv()->DeleteWeakGlobalRef(_jobj);
 
@@ -616,13 +625,23 @@ void JNI::enableZoning(JNIEnv *env, jobject self, jboolean enable) {
 	_system->enableZoning(enable);
 }
 
-void JNI::pauseEngine(JNIEnv *env, jobject self, jboolean pause) {
-	if (!_system || !g_engine)
+void JNI::pauseEngine(JNIEnv *env, jobject self, jboolean value) {
+	if (!_system)
 		return;
 
-	if ((pause && !g_engine->isPaused()) || (!pause && g_engine->isPaused())) {
-		LOGD("pauseEngine: %d", pause);
-		g_engine->pauseEngine(pause);
+	if (g_engine)
+		if ((value && !g_engine->isPaused()) ||
+				(!value && g_engine->isPaused())) {
+			LOGD("pauseEngine: %d", value);
+			g_engine->pauseEngine(value);
+		}
+
+	pause = value;
+
+	if (!pause) {
+		// wake up all threads
+		for (uint i = 0; i < 3; ++i)
+			sem_post(&pause_sem);
 	}
 }
 
-- 
cgit v1.2.3