tweaks

onthegomap · Oct 27, 2023 · 6f0ac9f · 6f0ac9f
1 parent 507ea4e
commit 6f0ac9f
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 6 deletions.
diff --git a/planetiler-core/src/main/java/com/onthegomap/planetiler/VectorTile.java b/planetiler-core/src/main/java/com/onthegomap/planetiler/VectorTile.java
@@ -516,7 +516,7 @@ public VectorTileProto.Tile toProto() {
           case Float floatValue -> tileValue.setFloatValue(floatValue);
           case Double doubleValue -> tileValue.setDoubleValue(doubleValue);
           case Boolean booleanValue -> tileValue.setBoolValue(booleanValue);
-          case Object ignored -> tileValue.setStringValue(value.toString());
+          case Object other -> tileValue.setStringValue(other.toString());
         }
         tileLayer.addValues(tileValue.build());
       }

diff --git a/planetiler-core/src/main/java/com/onthegomap/planetiler/collection/FeatureGroup.java b/planetiler-core/src/main/java/com/onthegomap/planetiler/collection/FeatureGroup.java
@@ -217,17 +217,17 @@ private byte[] encodeValue(VectorTile.Feature vectorTileFeature, RenderedFeature
       var attrs = vectorTileFeature.attrs();
       packer.packMapHeader((int) attrs.values().stream().filter(Objects::nonNull).count());
       for (Map.Entry<String, Object> entry : attrs.entrySet()) {
-        if (entry.getValue() != null) {
+        Object value = entry.getValue();
+        if (value != null) {
           packer.packInt(commonValueStrings.encode(entry.getKey()));
-          Object value = entry.getValue();
           packer.packValue(switch (value) {
             case String string -> ValueFactory.newString(string);
             case Integer integer -> ValueFactory.newInteger(integer.longValue());
             case Long longValue -> ValueFactory.newInteger(longValue);
             case Float floatValue -> ValueFactory.newFloat(floatValue);
             case Double doubleValue -> ValueFactory.newFloat(doubleValue);
             case Boolean booleanValue -> ValueFactory.newBoolean(booleanValue);
-            default -> ValueFactory.newString(value.toString());
+            case Object other -> ValueFactory.newString(other.toString());
           });
         }
       }

diff --git a/planetiler-core/src/main/java/com/onthegomap/planetiler/util/TileSizeStats.java b/planetiler-core/src/main/java/com/onthegomap/planetiler/util/TileSizeStats.java
@@ -44,7 +44,7 @@
  * archive, or run the following an existing archive:
  *
  * <pre>
- * {@snippet lang="bash" :
+ * {@code
  * java -jar planetiler.jar stats --input=<path to pmtiles or mbtiles> --output=layerstats.tsv.gz
  * }
  * </pre>

diff --git a/planetiler-core/src/main/java/com/onthegomap/planetiler/worker/WorkerPipeline.java b/planetiler-core/src/main/java/com/onthegomap/planetiler/worker/WorkerPipeline.java
@@ -17,7 +17,6 @@
  * A mini-framework for chaining sequential steps that run in dedicated threads with a queue between each.
  * <p>
  * For example:
- *
  * {@snippet :
  * WorkerPipeline.start("name", stats)
  *   .readFrom("reader", List.of(1, 2, 3))