I don't think we want people to subclass this API. Made it final to prevent mistakes.

Author: bytekeeper

src/main/java/bwem/Altitude.java

@@ -23,7 +23,7 @@ public final class Altitude implements Comparable<Altitude> {
     public static final Altitude ZERO = new Altitude(0);
     private final int val;
 
-    public Altitude(final int val) {
+    Altitude(final int val) {
         this.val = val;
     }
 

src/main/java/bwem/AreaId.java

@@ -21,7 +21,7 @@ public final class AreaId implements Comparable<AreaId> {
     public static final AreaId ZERO = new AreaId(0);
     private final int val;
 
-    public AreaId(final int val) {
+    AreaId(final int val) {
         this.val = val;
     }
 

src/main/java/bwem/AreaInitializer.java

@@ -25,7 +25,7 @@
 
 import static bwem.AreaId.UNINITIALIZED;
 
-class AreaInitializer extends Area {
+final class AreaInitializer extends Area {
 
     private static final StaticMarkable staticMarkable = new StaticMarkable();
     private final Markable markable;

src/main/java/bwem/Base.java

@@ -28,7 +28,7 @@
  * Base always belongs to some Area. An Area may contain zero, one or several Bases. Like Areas and
  * ChokePoints, the number and the addresses of Base instances remain unchanged.
  */
-public class Base {
+public final class Base {
     private final Area area;
     private final List<Mineral> minerals = new ArrayList<>();
     private final List<Geyser> geysers = new ArrayList<>();

src/main/java/bwem/CPPath.java

@@ -22,7 +22,7 @@
  * <p>cp.h:68:typedef std::vector<const ChokePoint *> Path; cp.h:168:typedef ChokePoint::Path
  * CPPath;
  */
-public class CPPath implements Iterable<ChokePoint> {
+public final class CPPath implements Iterable<ChokePoint> {
 
     static final CPPath EMPTY_PATH = new CPPath();
     private final List<ChokePoint> chokepoints;

src/main/java/bwem/ChokePoint.java

@@ -39,7 +39,7 @@
  * ChokePoint for each blocking Neutral (only one in the case of stacked blocking Neutral).<br>
  * Such ChokePoints are called pseudo ChokePoints and they behave differently in several ways.
  */
-public class ChokePoint {
+public final class ChokePoint {
     private final Graph graph;
     private final boolean isPseudo;
     private final int index;

src/main/java/bwem/Graph.java

@@ -35,7 +35,7 @@ public final class Graph {
             new ArrayList<>(); // index == ChokePoint::index x ChokePoint::index
     private final List<Base> bases = new ArrayList<>();
 
-    public Graph(BWMap map) {
+    Graph(BWMap map) {
         this.map = map;
     }
 

src/main/java/bwem/MapData.java

@@ -19,14 +19,14 @@
 import java.util.ArrayList;
 import java.util.List;
 
-public class MapData {
+public final class MapData {
     private final TilePosition tileSize;
     private final WalkPosition walkSize;
     private final Position pixelSize;
     private final Position center;
     private final List<TilePosition> startingLocations;
 
-    public MapData(
+    MapData(
             final int tileWidth, final int tileHeight, final List<TilePosition> startingLocations) {
 
         this.tileSize = new TilePosition(tileWidth, tileHeight);

src/main/java/bwem/MiniTile.java

@@ -28,7 +28,7 @@
  * - The whole process of analysis of a BWMap relies on the walkability information<br>
  * from which are derived successively: altitudes, Areas, ChokePoints.
  */
-public class MiniTile {
+public final class MiniTile {
     private static final AreaId blockingCP = new AreaId(Integer.MIN_VALUE);
 
     private Altitude
@@ -38,7 +38,7 @@ public class MiniTile {
             areaId; // 0 -> unwalkable  ;  > 0 -> index of some Area  ;  < 0 -> some walkable terrain, but
     // too small to be part of an Area
 
-    public MiniTile() {
+    MiniTile() {
         this.altitude = Altitude.UNINITIALIZED;
         this.areaId = UNINITIALIZED;
     }

src/main/java/bwem/NeutralData.java

@@ -17,16 +17,16 @@
 import java.util.ArrayList;
 import java.util.List;
 
-public class NeutralData {
+public final class NeutralData {
     private final List<Mineral> minerals;
     private final List<Geyser> geysers;
     private final List<StaticBuilding> staticBuildings;
 
-    public NeutralData(
-            final BWMap map,
-            final List<Unit> mineralPatches,
-            final List<Unit> vespeneGeysers,
-            final List<Unit> neutralUnits) {
+    NeutralData(
+        final BWMap map,
+        final List<Unit> mineralPatches,
+        final List<Unit> vespeneGeysers,
+        final List<Unit> neutralUnits) {
         ////////////////////////////////////////////////////////////////////////
         // BWMap::InitializeNeutrals
         ////////////////////////////////////////////////////////////////////////