yetangitu/NewPipe
1
0
Fork 0
mirror of https://github.com/TeamNewPipe/NewPipe.git synced 2025-10-03 09:49:21 +02:00
Code Issues Releases Wiki Activity

Player/handleIntent: move prefs parameters into initPlayback

Browse source 
They are just read from the player preferences and don’t influence the
branching, no need to read them in the intent parsing logic.
This commit is contained in:
Profpatsch 2025-05-05 15:48:26 +02:00
parent d534946550
commit 25a4a9a253
1 changed files with 8 additions and 18 deletions
Download patch file Download diff file Expand all files Collapse all files

26
app/src/main/java/org/schabi/newpipe/player/Player.java
View file

@ -385,12 +385,6 @@ public final class Player implements PlaybackListener, Listener {
}
}
// initPlayback Parameters
final PlaybackParameters savedParameters = retrievePlaybackParametersFromPrefs(this);
final float playbackSpeed = savedParameters.speed;
final float playbackPitch = savedParameters.pitch;
final boolean playbackSkipSilence = getPrefs().getBoolean(getContext().getString(
R.string.playback_skip_silence_key), getPlaybackSkipSilence());
final boolean playWhenReady = intent.getBooleanExtra(PLAY_WHEN_READY, true);
// branching parameters for below
@ -452,28 +446,24 @@ public final class Player implements PlaybackListener, Listener {
newQueue.setRecovery(newQueue.getIndex(),
state.getProgressMillis());
}
initPlayback(newQueue, playbackSpeed, playbackPitch,
playbackSkipSilence, playWhenReady);
initPlayback(newQueue, playWhenReady);
},
error -> {
if (DEBUG) {
Log.w(TAG, "Failed to start playback", error);
}
// In case any error we can start playback without history
initPlayback(newQueue, playbackSpeed, playbackPitch,
playbackSkipSilence, playWhenReady);
initPlayback(newQueue, playWhenReady);
},
() -> {
// Completed but not found in history
initPlayback(newQueue, playbackSpeed, playbackPitch,
playbackSkipSilence, playWhenReady);
initPlayback(newQueue, playWhenReady);
}
));
} else {
// Good to go...
// In a case of equal PlayQueues we can re-init old one but only when it is disposed
initPlayback(samePlayQueue ? playQueue : newQueue, playbackSpeed,
playbackPitch, playbackSkipSilence, playWhenReady);
initPlayback(samePlayQueue ? playQueue : newQueue, playWhenReady);
}
if (oldPlayerType != playerType && playQueue != null) {
@ -519,13 +509,13 @@ public final class Player implements PlaybackListener, Listener {
}
private void initPlayback(@NonNull final PlayQueue queue,
final float playbackSpeed,
final float playbackPitch,
final boolean playbackSkipSilence,
final boolean playOnReady) {
destroyPlayer();
initPlayer(playOnReady);
setPlaybackParameters(playbackSpeed, playbackPitch, playbackSkipSilence);
final boolean playbackSkipSilence = getPrefs().getBoolean(getContext().getString(
R.string.playback_skip_silence_key), getPlaybackSkipSilence());
final PlaybackParameters savedParameters = retrievePlaybackParametersFromPrefs(this);
setPlaybackParameters(savedParameters.speed, savedParameters.pitch, playbackSkipSilence);
playQueue = queue;
playQueue.init();