From b44acdd519fb42a73d76376c4c83b7898c443f7f Mon Sep 17 00:00:00 2001
From: Christian Schabesberger <chris.schabesberger@mailbox.org>
Date: Wed, 23 Oct 2024 19:31:21 +0200
Subject: [PATCH] remove unused saveInstanceState and add check to timeline
 change

---
 .../src/main/java/net/newpipe/newplayer/NewPlayerImpl.kt | 9 ++++++---
 .../newpipe/newplayer/uiModel/NewPlayerViewModelImpl.kt  | 3 ---
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/new-player/src/main/java/net/newpipe/newplayer/NewPlayerImpl.kt b/new-player/src/main/java/net/newpipe/newplayer/NewPlayerImpl.kt
index 6bc0776..39746d7 100644
--- a/new-player/src/main/java/net/newpipe/newplayer/NewPlayerImpl.kt
+++ b/new-player/src/main/java/net/newpipe/newplayer/NewPlayerImpl.kt
@@ -31,6 +31,7 @@ import androidx.media3.common.C
 import androidx.media3.common.MediaItem
 import androidx.media3.common.PlaybackException
 import androidx.media3.common.Player
+import androidx.media3.common.Player.TIMELINE_CHANGE_REASON_PLAYLIST_CHANGED
 import androidx.media3.common.Timeline
 import androidx.media3.common.Tracks
 import androidx.media3.common.util.UnstableApi
@@ -202,9 +203,11 @@ class NewPlayerImpl(
 
             override fun onTimelineChanged(timeline: Timeline, reason: Int) {
                 // TODO shouldn't you check that reason == TIMELINE_CHANGE_REASON_PLAYLIST_CHANGED
-                mutablePlaylist.update {
-                    (0..<newExoPlayer.mediaItemCount).map {
-                        newExoPlayer.getMediaItemAt(it)
+                if(reason == TIMELINE_CHANGE_REASON_PLAYLIST_CHANGED) {
+                    mutablePlaylist.update {
+                        (0..<newExoPlayer.mediaItemCount).map {
+                            newExoPlayer.getMediaItemAt(it)
+                        }
                     }
                 }
             }
diff --git a/new-player/src/main/java/net/newpipe/newplayer/uiModel/NewPlayerViewModelImpl.kt b/new-player/src/main/java/net/newpipe/newplayer/uiModel/NewPlayerViewModelImpl.kt
index 156319f..9446264 100644
--- a/new-player/src/main/java/net/newpipe/newplayer/uiModel/NewPlayerViewModelImpl.kt
+++ b/new-player/src/main/java/net/newpipe/newplayer/uiModel/NewPlayerViewModelImpl.kt
@@ -64,9 +64,6 @@ private const val TAG = "VideoPlayerViewModel"
 @UnstableApi
 @HiltViewModel
 class NewPlayerViewModelImpl @Inject constructor(
-    // TODO this seems unused? If you planned to save things inside here, explain your plans so they
-    //  can be implemented by future contributors
-    private val savedStateHandle: SavedStateHandle,
     application: Application,
 ) : AndroidViewModel(application), InternalNewPlayerViewModel {