Allowing GC

Author: devgianlu

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

@@ -310,6 +310,9 @@ void seek(int positionMs) {
 
     void stop() {
         stopped = true;
+        synchronized (pauseLock) {
+            pauseLock.notifyAll(); // Allow thread to exit
+        }
     }
 
     @Nullable

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

@@ -67,6 +67,7 @@ private void load(@NotNull TrackId id, boolean play, int pos) throws IOException
     }
 
     private void sendCommand(@NotNull Command command, Object... args) {
+        if (stopped) throw new IllegalStateException("Looper is stopped!");
         commands.add(new CommandBundle(command, args));
     }
 
@@ -119,6 +120,7 @@ PlayerRunner.Controller controller() {
     public void close() {
         stopped = true;
         if (playerRunner != null) playerRunner.stop();
+        commands.add(new CommandBundle(Command.Terminate));
     }
 
     @Nullable
@@ -132,7 +134,7 @@ boolean isTrack(@NotNull TrackId id) {
 
     public enum Command {
         Load, Play, Pause,
-        Stop, Seek
+        Stop, Seek, Terminate
     }
 
     public interface Listener {
@@ -175,6 +177,8 @@ public void run() {
                         case Seek:
                             if (playerRunner != null) playerRunner.seek((Integer) cmd.args[0]);
                             break;
+                        case Terminate:
+                            return;
                     }
                 }
             } catch (InterruptedException ex) {