From 189a0c658b7d8330e8bee07a1b02cad327e14bde Mon Sep 17 00:00:00 2001 From: Kingkor Roy Tirtho Date: Mon, 20 Jul 2026 18:28:58 +0600 Subject: [PATCH] feat: simplify playback control logic in AudioPlayer by removing redundant state checks --- .../spotube/core/audioplayer/AudioPlayer.jvm.kt | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/composeApp/src/jvmMain/kotlin/dev/krtirtho/spotube/core/audioplayer/AudioPlayer.jvm.kt b/composeApp/src/jvmMain/kotlin/dev/krtirtho/spotube/core/audioplayer/AudioPlayer.jvm.kt index e06ecda9..905cf5e2 100644 --- a/composeApp/src/jvmMain/kotlin/dev/krtirtho/spotube/core/audioplayer/AudioPlayer.jvm.kt +++ b/composeApp/src/jvmMain/kotlin/dev/krtirtho/spotube/core/audioplayer/AudioPlayer.jvm.kt @@ -306,10 +306,7 @@ actual class AudioPlayer actual constructor(context: Any) : KoinComponent { lock.withLock { if (disposed || currentPlaylist.isEmpty() || shuffleEnabled == enabled) return - val wasPlaying = _playerState.value == PlayerState.PLAYING - val wasPaused = _playerState.value == PlayerState.PAUSED val currentItem = _currentMediaItem.value - val currentTimeMs = mediaPlayer.status().time().coerceAtLeast(0) val rebuilt = if (enabled) { if (currentItem != null) { @@ -338,16 +335,6 @@ actual class AudioPlayer actual constructor(context: Any) : KoinComponent { currentIndex = nextIndex _currentMediaItem.tryEmit(currentPlaylist.getOrNull(nextIndex)) - - if (nextIndex >= 0 && (wasPlaying || wasPaused)) { - mediaListPlayer.controls().play(nextIndex) - if (currentTimeMs > 0) { - mediaPlayer.controls().setTime(currentTimeMs) - } - if (wasPaused) { - mediaListPlayer.controls().setPause(true) - } - } } }