From 0245f926d12174261a77dd8fba9636ad2382251c Mon Sep 17 00:00:00 2001
From: twinaphex <libretro@gmail.com>
Date: Fri, 13 Sep 2013 15:57:13 +0200
Subject: [PATCH] (Android Phoenix) Set configurable audio latency

---
 android/phoenix/res/values/array.xml          | 12 ++++++++
 android/phoenix/res/xml/prefs.xml             | 12 ++++----
 .../retroarch/browser/MainMenuActivity.java   | 28 ++++++++++++++-----
 3 files changed, 40 insertions(+), 12 deletions(-)

diff --git a/android/phoenix/res/values/array.xml b/android/phoenix/res/values/array.xml
index 823710ca5f..6fc8d82643 100644
--- a/android/phoenix/res/values/array.xml
+++ b/android/phoenix/res/values/array.xml
@@ -10,6 +10,18 @@
         <item>1</item>
     </string-array>
 
+    <string-array name="audio_latency_options">
+        <item> 64ms (Low)</item>
+        <item>128ms (Medium)</item>
+        <item>160ms (High)</item>
+    </string-array>
+
+    <string-array name="audio_latency_options_values">
+       <item>64</item>
+       <item>128</item>
+       <item>160</item>
+    </string-array>
+
     <string-array name="aspect_ratios">
         <item>Full screen</item>
         <item>Auto</item>
diff --git a/android/phoenix/res/xml/prefs.xml b/android/phoenix/res/xml/prefs.xml
index 7be204e611..91a0b3c118 100644
--- a/android/phoenix/res/xml/prefs.xml
+++ b/android/phoenix/res/xml/prefs.xml
@@ -114,11 +114,13 @@
                 android:summary="Enable dynamic rate control (recommended)."
                 android:title="Dynamic Rate Control"
                 android:dependency="audio_enable" />
-            <CheckBoxPreference 
-                android:defaultValue="false"
-                android:key="audio_high_latency"
-                android:summary="Use (very) high latency audio. Necessary for older Android devices with poor audio drivers."
-                android:title="High latency audio"
+            <ListPreference
+                android:entries="@array/audio_latency_options"
+                android:entryValues="@array/audio_latency_options_values"
+                android:defaultValue="64"
+                android:key="audio_latency"
+                android:summary="Set the amount of desired audio latency. Your mileage may vary on the device you are using."
+                android:title="Audio latency"
                 android:dependency="audio_enable" />
             <CheckBoxPreference 
                 android:defaultValue="true"
diff --git a/android/phoenix/src/org/retroarch/browser/MainMenuActivity.java b/android/phoenix/src/org/retroarch/browser/MainMenuActivity.java
index 710f799649..31e8d48ccf 100644
--- a/android/phoenix/src/org/retroarch/browser/MainMenuActivity.java
+++ b/android/phoenix/src/org/retroarch/browser/MainMenuActivity.java
@@ -366,17 +366,22 @@ public class MainMenuActivity extends PreferenceActivity {
 			boolean lowLatency = hasLowLatencyAudio();
 			Log.i(TAG, "Audio is low latency: " + (lowLatency ? "yes" : "no"));
 			
-			if (lowLatency && !prefs.getBoolean("audio_high_latency", false)) {
-				config.setInt("audio_latency", 64);
+			config.setInt("audio_latency", 64);
+			if (lowLatency) {
 				config.setInt("audio_block_frames", buffersize);
 			} else {
-				config.setInt("audio_latency", prefs.getBoolean(
-						"audio_high_latency", false) ? 160 : 64);
 				config.setInt("audio_block_frames", 0);
 			}
 		} else {
-			config.setInt("audio_latency",
-					prefs.getBoolean("audio_high_latency", false) ? 160 : 64);		
+			String latency_audio = prefs.getString("audio_latency", "64");
+			
+			if (latency_audio.equals("64")) {
+				config.setInt("audio_latency", 64);
+			} else if (latency_audio.equals("128")) {
+				config.setInt("audio_latency", 128);
+			} else if (latency_audio.equals("160")) {
+				config.setInt("audio_latency", 160);
+			}
 		}
 
 		config.setBoolean("audio_enable",
@@ -682,6 +687,8 @@ public class MainMenuActivity extends PreferenceActivity {
 											false);
 									edit.putBoolean("input_autodetect_enable",
 											true);
+									edit.putString("audio_latency", "64");
+									edit.putBoolean("audio_latency_auto", true);
 									edit.commit();
 								}
 							});
@@ -703,7 +710,8 @@ public class MainMenuActivity extends PreferenceActivity {
 											false);
 									edit.putBoolean("input_autodetect_enable",
 											true);
-									edit.putBoolean("audio_high_latency", true);
+									edit.putString("audio_latency", "160");
+									edit.putBoolean("audio_latency_auto", false);
 									edit.commit();
 								}
 							});
@@ -725,6 +733,8 @@ public class MainMenuActivity extends PreferenceActivity {
 											false);
 									edit.putBoolean("input_autodetect_enable",
 											true);
+									edit.putString("audio_latency", "64");
+									edit.putBoolean("audio_latency_auto", true);
 									edit.commit();
 								}
 							});
@@ -749,6 +759,8 @@ public class MainMenuActivity extends PreferenceActivity {
 													true);
 											edit.putString("video_refresh_rate", Double
 													.valueOf(59.19132938771038).toString());
+											edit.putString("audio_latency", "128");
+											edit.putBoolean("audio_latency_auto", false);
 											edit.commit();
 										}
 									});
@@ -768,6 +780,8 @@ public class MainMenuActivity extends PreferenceActivity {
 											.edit();
 									edit.putString("video_refresh_rate", Double
 											.valueOf(59.65).toString());
+									edit.putString("audio_latency", "64");
+									edit.putBoolean("audio_latency_auto", false);
 									edit.commit();
 								}
 							});