From dda3677b001647aaee2dd35dcc0e12bdf1fad27f Mon Sep 17 00:00:00 2001
From: sh123 <sh123@users.noreply.github.com>
Date: Mon, 25 Dec 2023 12:51:56 +0200
Subject: [PATCH] Increase version, do not use super frame size for OPUS

---
 codec2talkie/build.gradle                                | 4 ++--
 .../java/com/radio/codec2talkie/protocol/AudioOpus.java  | 7 ++-----
 codec2talkie/src/main/res/values/strings.xml             | 2 +-
 codec2talkie/src/main/res/xml/preferences.xml            | 9 +--------
 codec2talkie/src/main/res/xml/preferences_codec.xml      | 7 +++++++
 5 files changed, 13 insertions(+), 16 deletions(-)

diff --git a/codec2talkie/build.gradle b/codec2talkie/build.gradle
index 7e440cc..4764782 100644
--- a/codec2talkie/build.gradle
+++ b/codec2talkie/build.gradle
@@ -10,8 +10,8 @@ android {
         applicationId "com.radio.codec2talkie"
         minSdkVersion 23
         targetSdkVersion 30
-        versionCode 168
-        versionName "1.68"
+        versionCode 169
+        versionName "1.69"
 
         testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
     }
diff --git a/codec2talkie/src/main/java/com/radio/codec2talkie/protocol/AudioOpus.java b/codec2talkie/src/main/java/com/radio/codec2talkie/protocol/AudioOpus.java
index 986492e..58b6f2c 100644
--- a/codec2talkie/src/main/java/com/radio/codec2talkie/protocol/AudioOpus.java
+++ b/codec2talkie/src/main/java/com/radio/codec2talkie/protocol/AudioOpus.java
@@ -2,7 +2,6 @@ package com.radio.codec2talkie.protocol;
 
 import android.content.Context;
 import android.content.SharedPreferences;
-import android.text.TextUtils;
 import android.util.Log;
 
 import androidx.preference.PreferenceManager;
@@ -10,7 +9,6 @@ import androidx.preference.PreferenceManager;
 import com.radio.codec2talkie.protocol.message.TextMessage;
 import com.radio.codec2talkie.protocol.position.Position;
 import com.radio.codec2talkie.settings.PreferenceKeys;
-import com.radio.codec2talkie.tools.DebugTools;
 import com.radio.codec2talkie.transport.Transport;
 import com.radio.opus.Opus;
 
@@ -48,18 +46,17 @@ public class AudioOpus implements Protocol {
         int complexity = Integer.parseInt(sharedPreferences.getString(PreferenceKeys.OPUS_COMPLEXITY, "5"));
         float pcmFrameDuration = Float.parseFloat(sharedPreferences.getString(PreferenceKeys.OPUS_FRAME_SIZE, "40"));
 
-        int superFrameSize = Integer.parseInt(sharedPreferences.getString(PreferenceKeys.CODEC2_TX_FRAME_MAX_SIZE, "48"));
         _pcmFrameSize = (int)(SAMPLE_RATE / 1000 * pcmFrameDuration);
         _audioBufferSize = 10*_pcmFrameSize;
 
         _playbackAudioBuffer = new short[_audioBufferSize];
-        _recordAudioEncodedBuffer = new byte[superFrameSize];
+        _recordAudioEncodedBuffer = new byte[_audioBufferSize];
 
         _opusCon = Opus.create(SAMPLE_RATE, 1, Opus.OPUS_APPLICATION_VOIP, bitRate, complexity);
         if (_opusCon == 0) {
             Log.e(TAG, "Failed to create opus");
         }
-        Log.i(TAG, "Opus is initialized, pcm frame size: " + _pcmFrameSize + ", super frame size: " + superFrameSize);
+        Log.i(TAG, "Opus is initialized, pcm frame size: " + _pcmFrameSize + ", buffer size: " + _audioBufferSize);
     }
 
     @Override
diff --git a/codec2talkie/src/main/res/values/strings.xml b/codec2talkie/src/main/res/values/strings.xml
index 8127d66..21b030c 100644
--- a/codec2talkie/src/main/res/values/strings.xml
+++ b/codec2talkie/src/main/res/values/strings.xml
@@ -180,7 +180,7 @@
     <string name="kiss_toast_modem_reboot">Modem reboot requested</string>
 
     <string name="codec2_tx_frame_max_size_title">Maximum super frame size (bytes)</string>
-    <string name="codec2_tx_frame_max_size_summary">Multiple audio codec encoded samples are aggregated into super frame not larger than this value</string>
+    <string name="codec2_tx_frame_max_size_summary">Multiple Codec2 encoded samples are aggregated into super frame not larger than this value</string>
 
     <string name="usb_settings_title">USB settings</string>
     <string name="usb_data_bits_title">Serial data bits</string>
diff --git a/codec2talkie/src/main/res/xml/preferences.xml b/codec2talkie/src/main/res/xml/preferences.xml
index 7f6e23c..d30d558 100644
--- a/codec2talkie/src/main/res/xml/preferences.xml
+++ b/codec2talkie/src/main/res/xml/preferences.xml
@@ -133,14 +133,7 @@
             app:summary="@string/codec_type_settings_summary"
             app:fragment="com.radio.codec2talkie.settings.SettingsActivity$SettingsCodecFragment">
         </Preference>
-
-        <EditTextPreference
-            app:key="codec2_tx_frame_max_size"
-            app:title="@string/codec2_tx_frame_max_size_title"
-            app:summary="@string/codec2_tx_frame_max_size_summary"
-            app:defaultValue="48">
-        </EditTextPreference>
-
+        
         <SwitchPreference
             app:key="codec2_recording_enabled"
             app:title="@string/codec2_recorder_title"
diff --git a/codec2talkie/src/main/res/xml/preferences_codec.xml b/codec2talkie/src/main/res/xml/preferences_codec.xml
index d0354ef..a4c8b93 100644
--- a/codec2talkie/src/main/res/xml/preferences_codec.xml
+++ b/codec2talkie/src/main/res/xml/preferences_codec.xml
@@ -15,6 +15,13 @@
             app:summary="%s">
         </ListPreference>
 
+        <EditTextPreference
+            app:key="codec2_tx_frame_max_size"
+            app:title="@string/codec2_tx_frame_max_size_title"
+            app:summary="@string/codec2_tx_frame_max_size_summary"
+            app:defaultValue="48">
+        </EditTextPreference>
+
     </PreferenceCategory>
 
     <PreferenceCategory