Removed unnecessary allocations in Game API.

Author: dgant

src/main/java/bwapi/BWClient.java

@@ -17,7 +17,7 @@ public class BWClient {
     public BWClient(final BWEventListener eventListener) {
         Objects.requireNonNull(eventListener);
         this.eventListener = eventListener;
-        this.timerResolutionThread = new TimerResolutionThread();
+        //this.timerResolutionThread = new TimerResolutionThread();
     }
 
     /**
@@ -81,7 +81,7 @@ public void startGame(BWClientConfiguration configuration) {
 
         // Use reduced priority to encourage Windows to give priority to StarCraft.exe/BWAPI.
         // If BWAPI doesn't get priority, it may not detect completion of a frame on our end in timely fashion.
-        Thread.currentThread().setPriority(Thread.NORM_PRIORITY);
+        Thread.currentThread().setPriority(4);
 
         if (client == null) {
             client = new Client(configuration);

src/main/java/bwapi/BotWrapper.java

@@ -82,7 +82,7 @@ void onFrame() {
                 botThread = createBotThread();
                 botThread.setName("JBWAPI Bot");
                 // Reduced priority helps ensure that StarCraft.exe/BWAPI pick up on our frame completion in timely fashion
-                botThread.setPriority(Thread.MIN_PRIORITY);
+                botThread.setPriority(3);
                 botThread.start();
             }
 

src/main/java/bwapi/Client.java

@@ -247,9 +247,13 @@ void update() {
         while (code != 2) {
             try {
                 code = pipeObjectHandle.readByte();
+                /*
                 if (code != 2) {
-                    Thread.sleep(0, 1);
+                    try {
+                        Thread.sleep(0, 1);
+                    } catch (InterruptedException ignored) {}
                 }
+                */
             }
             catch (Exception e) {
                 System.err.println("failed, disconnecting");