sylar/PianoTimelineApp
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fixed small bug in cached play offset

Browse source
This commit is contained in:
OmniaX-Dev 2026-07-06 06:25:52 +02:00
parent 35b77c75fe
commit a16f00a42b
1 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
app/src/main/java/com/keylightpiano/pianotimeline/ui/RecordingListViewModel.kt
View file

@ -272,11 +272,15 @@ class RecordingListViewModel(app: Application) : AndroidViewModel(app) {
} }
// Throttled persistence of the current track + offset: at most once per // Throttled persistence of the current track + offset: at most once per
// second, and only while a track is loaded. Saving live (every 200ms // second, and only while a track is actively PLAYING. Saving while paused
// position tick) would hammer DataStore; 1s is plenty for resume. // is both unnecessary (a paused position doesn't change) and harmful: on
// restore, preparePaused seeks asynchronously, so ExoPlayer briefly reports
// position 0 before the seek lands — saving during that window would clobber
// the good offset with 0. Gating on isPlaying closes that window.
viewModelScope.launch { viewModelScope.launch {
var lastSaveAt = 0L var lastSaveAt = 0L
playbackState.collect { s -> playbackState.collect { s ->
if (!s.isPlaying) return@collect
val id = s.nowPlayingId ?: return@collect val id = s.nowPlayingId ?: return@collect
val now = System.currentTimeMillis() val now = System.currentTimeMillis()
if (now - lastSaveAt >= 1000L) { if (now - lastSaveAt >= 1000L) {