Fixed pausing state being overridden by halt listener

Author: devgianlu

core/src/main/java/xyz/gianlu/librespot/player/Player.java

@@ -231,7 +231,7 @@ public void notActive() {
     @Override
     public void startedLoading(@NotNull TrackHandler handler) {
         if (handler == trackHandler) {
-            state.setState(true, null, true);
+            state.setBuffering(true);
             state.updated();
         }
     }
@@ -249,7 +249,7 @@ private void updateStateWithHandler() {
     @Override
     public void finishedLoading(@NotNull TrackHandler handler, int pos) {
         if (handler == trackHandler) {
-            state.setState(true, null, false);
+            state.setBuffering(false);
 
             updateStateWithHandler();
 
@@ -368,7 +368,7 @@ public void playbackHalted(@NotNull TrackHandler handler, int chunk) {
         if (handler == trackHandler) {
             LOGGER.debug(String.format("Playback halted on retrieving chunk %d.", chunk));
 
-            state.setState(true, false, true);
+            state.setBuffering(true);
             state.updated();
 
             events.playbackHaltStateChanged(true);
@@ -381,7 +381,7 @@ public void playbackResumedFromHalt(@NotNull TrackHandler handler, int chunk, lo
             LOGGER.debug(String.format("Playback resumed, chunk %d retrieved, took %dms.", chunk, diff));
 
             state.setPosition(state.getPosition() - diff);
-            state.setState(true, false, false);
+            state.setBuffering(false);
             state.updated();
 
             events.playbackHaltStateChanged(false);

core/src/main/java/xyz/gianlu/librespot/player/StateWrapper.java

@@ -96,9 +96,8 @@ private static boolean shouldPlay(@NotNull ContextTrack track) {
         return PlayableId.isSupported(track.getUri()) && PlayableId.shouldPlay(track);
     }
 
-    void setState(@Nullable Boolean playing, @Nullable Boolean paused, @Nullable Boolean buffering) {
-        setState(playing == null ? state.getIsPlaying() : playing, paused == null ? state.getIsPaused() : paused,
-                buffering == null ? state.getIsBuffering() : buffering);
+    void setBuffering(boolean buffering) {
+        setState(state.getIsPlaying(), state.getIsPaused(), buffering);
     }
 
     synchronized void setState(boolean playing, boolean paused, boolean buffering) {