audio resume/pause のログメッセージを修正, 分岐を when に置き換え

Author: t-miya

sora-android-sdk/src/main/kotlin/jp/shiguredo/sora/sdk/channel/rtc/AudioDeviceModuleWrapper.kt

@@ -23,7 +23,8 @@ import org.webrtc.audio.JavaAudioDeviceModule
  * 本クラスは専用の HandlerThread を用意し、そこで pauseRecording()/resumeRecording() を実行した上で
  * タイムアウト [PAUSE_TIMEOUT_MILLIS] ミリ秒で待機して結果を返します。
  * これは libwebrtc 側で録音スレッド停止の際にタイムアウト 2,000 ミリ秒の join 待ちが発生するためです。
- * また、ADM として JavaAudioDeviceModule 利用想定のため、他 ADM での動作はサポートしていません。
+ *
+ * また、アプリ側でカスタム ADM を差し込む場合は本クラスは利用されず、録音停止／再開はサポートされません。
  */
 class AudioDeviceModuleWrapper(
     private val adm: AudioDeviceModule,
@@ -80,6 +81,6 @@ class AudioDeviceModuleWrapper(
     private fun runCatchingWithLog(block: () -> Boolean): Boolean =
         runCatching(block)
             .onFailure {
-                SoraLogger.w(TAG, "pause/resume failed: ${it.message}")
+                SoraLogger.w(TAG, "pause/resume failed: ${it.message ?: it.javaClass.simpleName}", it)
             }.getOrDefault(false)
 }

sora-android-sdk/src/main/kotlin/jp/shiguredo/sora/sdk/channel/rtc/PeerChannel.kt

@@ -843,26 +843,19 @@ class PeerChannelImpl(
             return false
         }
         return withContext(executorDispatcher) {
-            if (paused) {
-                if (audioRecordingPaused) {
-                    true
-                } else {
-                    pauseAudioRecording()
-                }
-            } else {
-                if (!audioRecordingPaused) {
-                    true
-                } else {
-                    resumeAudioRecording()
-                }
+            when {
+                paused && audioRecordingPaused -> true
+                !paused && !audioRecordingPaused -> true
+                paused -> pauseAudioRecording()
+                else -> resumeAudioRecording()
             }
         }
     }
 
     /** ADM の録音を停止（マイクインジケータ消灯狙い） */
     private suspend fun pauseAudioRecording(): Boolean {
         val admWrapper = componentFactory.controllableAdm
-        SoraLogger.d(TAG, "[audio_recording_pause] pausing audio recording")
+        SoraLogger.d(TAG, "[audio_recording_pause] pausing ADM audio recording")
         // 既に停止中の場合でも true が返る
         val paused = admWrapper?.pauseRecording() ?: false
         SoraLogger.d(TAG, "[audio_recording_pause] pauseRecording result=$paused")
@@ -898,7 +891,7 @@ class PeerChannelImpl(
     private suspend fun resumeAudioRecording(): Boolean {
         // ADM の録音を再開
         val admWrapper = componentFactory.controllableAdm
-        SoraLogger.d(TAG, "[audio_recording_pause] resume audio recording")
+        SoraLogger.d(TAG, "[audio_recording_pause] resume ADM audio recording")
         // 既に録音中の場合でも true が返る
         val resumed = admWrapper?.resumeRecording() ?: false
         SoraLogger.d(TAG, "[audio_recording_pause] resumeRecording result=$resumed")