kitty/spotube
1
0
Fork 0
mirror of https://github.com/KRTirtho/spotube.git synced 2026-08-05 19:59:51 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

feat: simplify playback control logic in AudioPlayer by removing redundant state checks

Browse Source
This commit is contained in:
Kingkor Roy Tirtho 2026-07-20 18:28:58 +06:00
parent 9013b374c7
commit 189a0c658b
1 changed files with 0 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
composeApp/src/jvmMain/kotlin/dev/krtirtho/spotube/core/audioplayer/AudioPlayer.jvm.kt
View File

@ -306,10 +306,7 @@ actual class AudioPlayer actual constructor(context: Any) : KoinComponent {
lock.withLock { lock.withLock {
if (disposed || currentPlaylist.isEmpty() || shuffleEnabled == enabled) return if (disposed || currentPlaylist.isEmpty() || shuffleEnabled == enabled) return
val wasPlaying = _playerState.value == PlayerState.PLAYING
val wasPaused = _playerState.value == PlayerState.PAUSED
val currentItem = _currentMediaItem.value val currentItem = _currentMediaItem.value
val currentTimeMs = mediaPlayer.status().time().coerceAtLeast(0)
val rebuilt = if (enabled) { val rebuilt = if (enabled) {
if (currentItem != null) { if (currentItem != null) {
@ -338,16 +335,6 @@ actual class AudioPlayer actual constructor(context: Any) : KoinComponent {
currentIndex = nextIndex currentIndex = nextIndex
_currentMediaItem.tryEmit(currentPlaylist.getOrNull(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)
}
}
} }
} }