refactor: make listener suspendable

removes a call of launchCatchingIO

Author: BrayanDSO

AnkiDroid/src/main/java/com/ichi2/anki/cardviewer/CardMediaPlayer.kt

@@ -127,9 +127,9 @@ class CardMediaPlayer : Closeable {
     var playAvTagsJob: Job? = null
     val isPlaying get() = playAvTagsJob != null
 
-    private var onMediaGroupCompleted: (() -> Unit)? = null
+    private var onMediaGroupCompleted: (suspend () -> Unit)? = null
 
-    fun setOnMediaGroupCompletedListener(listener: (() -> Unit)?) {
+    fun setOnMediaGroupCompletedListener(listener: (suspend () -> Unit)?) {
         onMediaGroupCompleted = listener
     }
 

AnkiDroid/src/main/java/com/ichi2/anki/ui/windows/reviewer/ReviewerViewModel.kt

@@ -160,14 +160,12 @@ class ReviewerViewModel(
             updateNextTimes()
         }
         cardMediaPlayer.setOnMediaGroupCompletedListener {
-            launchCatchingIO {
-                if (!autoAdvance.shouldWaitForAudio()) return@launchCatchingIO
+            if (!autoAdvance.shouldWaitForAudio()) return@setOnMediaGroupCompletedListener
 
-                if (showingAnswer.value) {
-                    autoAdvance.onShowAnswer()
-                } else {
-                    autoAdvance.onShowQuestion()
-                }
+            if (showingAnswer.value) {
+                autoAdvance.onShowAnswer()
+            } else {
+                autoAdvance.onShowQuestion()
             }
         }
     }