misc cleanup 4

src/main/java/baritone/Settings.java

@@ -414,7 +414,7 @@ public class Settings {
                 }
             }
         } catch (IllegalAccessException e) {
-            throw new RuntimeException(e);
+            throw new IllegalStateException(e);
         }
         byLowerName = Collections.unmodifiableMap(tmpByName);
         allSettings = Collections.unmodifiableList(tmpAll);

src/main/java/baritone/cache/CachedRegion.java

@@ -100,7 +100,7 @@ public final class CachedRegion implements IBlockTypeAccess {
         return res;
     }
 
-    final synchronized void updateCachedChunk(int chunkX, int chunkZ, CachedChunk chunk) {
+    public final synchronized void updateCachedChunk(int chunkX, int chunkZ, CachedChunk chunk) {
         this.chunks[chunkX][chunkZ] = chunk;
         hasUnsavedChanges = true;
     }

src/main/java/baritone/pathing/calc/AStarPathFinder.java

@@ -116,7 +116,7 @@ public class AStarPathFinder extends AbstractNodeCostSearch implements Helper {
                 int chunkZ = currentNodePos.z >> 4;
                 if (dest.x >> 4 != chunkX || dest.z >> 4 != chunkZ) {
                     // only need to check if the destination is a loaded chunk if it's in a different chunk than the start of the movement
-                    if (chunkProvider.getLoadedChunk(chunkX, chunkZ) == null) {
+                    if (chunkProvider.isChunkGeneratedAt(chunkX, chunkZ)) {
                         // see issue #106
                         if (cachedWorld == null || !cachedWorld.isCached(dest)) {
                             numEmptyChunk++;

src/main/java/baritone/pathing/calc/openset/LinkedListOpenSet.java

@@ -83,7 +83,7 @@ class LinkedListOpenSet implements IOpenSet {
     }
 
     public static class Node { //wrapper with next
-        Node nextOpen;
-        PathNode val;
+        private Node nextOpen;
+        private PathNode val;
     }
 }