Minor tweaks

Author: dgant

src/main/java/bwapi/ClientData.java

@@ -1,6 +1,6 @@
 package bwapi;
 import java.nio.ByteBuffer;
-class ClientData {
+final class ClientData {
     private WrappedBuffer buffer;
     private GameData gameData;
     ClientData() {

src/main/java/bwapi/FrameBuffer.java

@@ -22,7 +22,6 @@ class FrameBuffer {
     private int stepBot = 0;
     private ArrayList<ByteBuffer> dataBuffer = new ArrayList<>();
 
-    // Synchronization locks
     private final Lock lockWrite = new ReentrantLock();
     final Lock lockSize = new ReentrantLock();
     final Condition conditionSize = lockSize.newCondition();

src/main/java/bwapi/Unit.java

@@ -1019,7 +1019,7 @@ public Order getOrder() {
 
     /**
      * Retrieves the secondary Order that the unit is assigned. Secondary
-     * orders are step in the background as a sub-order. An example would be {@link Order#TrainFighter},
+     * orders are run in the background as a sub-order. An example would be {@link Order#TrainFighter},
      * because a @Carrier can move and train fighters at the same time.
      *
      * @return The secondary {@link Order} that the unit is executing.

src/test/java/DumpToClient.java

@@ -91,7 +91,7 @@ public static void main(String[] args) throws IOException {
         try (PrintWriter out = new PrintWriter(sw)) {
             out.println("package bwapi;");
             out.println("import java.nio.ByteBuffer;");
-            out.println("class ClientData {");
+            out.println("final class ClientData {");
             out.println("    private WrappedBuffer buffer;");
             out.println("    private GameData gameData;");
             out.println("    ClientData() {");

src/test/java/bwapi/ClientDataBenchmark.java

@@ -19,9 +19,8 @@ public static class EmptyState {
 
         @Setup(Level.Invocation)
         public void setup() {
-            client = new Client(ByteBuffer.allocateDirect(ClientData.GameData.SIZE));
             game = new Game();
-            game.clientData().setBuffer(client.mapFile());
+            game.clientData().setBuffer(ByteBuffer.allocateDirect(ClientData.GameData.SIZE));
             strings = buildStrings();
         }
 
@@ -35,10 +34,9 @@ public static class FilledWithStrings {
 
         @Setup(Level.Invocation)
         public void setup() {
-            client = new Client(ByteBuffer.allocateDirect(ClientData.GameData.SIZE));
             data = client.clientData().gameData();
             game = new Game();
-            game.clientData().setBuffer(client.mapFile());
+            game.clientData().setBuffer(ByteBuffer.allocateDirect(ClientData.GameData.SIZE));
             String[] strings = buildStrings();
             for (String s : strings) {
                 GameDataUtils.addString(client.clientData().gameData(), s);