justinzeus/Wholphin
1
0
Fork 0
mirror of https://github.com/JustinZeus/Wholphin.git synced 2026-07-08 23:51:21 +02:00
Code Issues Projects Packages Wiki Activity Actions

Fix switching audio/subtitle tracks during playback (#121)

Browse source 
An error would occur when switching audio or subtitle tracks during
playback, but would be ignored. This meant the tracks can't be changed.

#111 was too aggressive on switching dispatchers. The player can only be
accessed on `Main`, so this PR fixes that.
This commit is contained in:
damontecres 2025-10-30 19:24:53 -04:00 • committed by GitHub
parent 982506555f
commit c65831cb05
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 6 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

9
app/src/main/java/com/github/damontecres/wholphin/ui/playback/PlaybackViewModel.kt
View file

@ -56,6 +56,7 @@ import com.github.damontecres.wholphin.util.subtitleMimeTypes
import com.github.damontecres.wholphin.util.supportItemKinds
import dagger.hilt.android.lifecycle.HiltViewModel
import dagger.hilt.android.qualifiers.ApplicationContext
import kotlinx.coroutines.CoroutineScope
import kotlinx.coroutines.Dispatchers
import kotlinx.coroutines.Job
import kotlinx.coroutines.delay
@ -590,7 +591,7 @@ class PlaybackViewModel
currentItemPlayback.value!!,
index,
currentItemPlayback.value?.subtitleIndex,
player.currentPosition,
onMain { player.currentPosition },
true,
)
}
@ -603,7 +604,7 @@ class PlaybackViewModel
currentItemPlayback.value!!,
currentItemPlayback.value?.audioIndex,
index,
player.currentPosition,
onMain { player.currentPosition },
true,
)
}
@ -656,7 +657,7 @@ class PlaybackViewModel
while (isActive) {
delay(500L)
val currentTicks =
withContext(Dispatchers.Main) { player.currentPosition.milliseconds.inWholeTicks }
onMain { player.currentPosition.milliseconds.inWholeTicks }
val currentSegment =
segments.items
.firstOrNull {
@ -956,6 +957,8 @@ val Format.idAsInt: Int?
}
}
suspend fun <T> onMain(block: suspend CoroutineScope.() -> T) = withContext(Dispatchers.Main, block)
@OptIn(UnstableApi::class)
private fun applyTrackSelections(
player: Player,