From 41230b5afbdec28bb9e9d6fc1399d2412ae227ce Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Thu, 4 Jun 2026 00:42:35 -0400
Subject: [PATCH] fix(android): single jump per polling loop for detekt
 LoopWithTooManyJumpStatements

---
 .../minstrel/player/PlayerController.kt           | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/android/app/src/main/java/com/fabledsword/minstrel/player/PlayerController.kt b/android/app/src/main/java/com/fabledsword/minstrel/player/PlayerController.kt
index 3f7ae5a8..74cc2dc2 100644
--- a/android/app/src/main/java/com/fabledsword/minstrel/player/PlayerController.kt
+++ b/android/app/src/main/java/com/fabledsword/minstrel/player/PlayerController.kt
@@ -524,13 +524,14 @@ class PlayerController @Inject constructor(
                 val changed = current.isPlaying != effectiveIsPlaying ||
                     current.positionMs != newPos ||
                     current.durationMs != newDur
-                if (!changed) continue
-                uiStateInternal.value = current.copy(
-                    isPlaying = effectiveIsPlaying,
-                    positionMs = newPos,
-                    durationMs = newDur,
-                    bufferedPositionMs = newBuf,
-                )
+                if (changed) {
+                    uiStateInternal.value = current.copy(
+                        isPlaying = effectiveIsPlaying,
+                        positionMs = newPos,
+                        durationMs = newDur,
+                        bufferedPositionMs = newBuf,
+                    )
+                }
             }
         }
     }