added BY_TEMP_HAS_NAME comparator to avoid its repeated construction …

…during run-time
openmaptiles · Dec 12, 2023 · d756038 · d756038
1 parent 612f700
commit d756038
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/main/java/org/openmaptiles/layers/Housenumber.java b/src/main/java/org/openmaptiles/layers/Housenumber.java
@@ -75,6 +75,8 @@ public class Housenumber implements
   private static final Pattern NO_CONVERSION_PATTERN = Pattern.compile("[^0-9;]");
   private static final String TEMP_PARTITION = "_partition";
   private static final String TEMP_HAS_NAME = "_has_name";
+  private static final Comparator<VectorTile.Feature> BY_TEMP_HAS_NAME = Comparator
+    .comparing(i -> (Boolean) i.attrs().get(TEMP_HAS_NAME), Boolean::compare);
   private final Stats stats;
 
   public Housenumber(Translations translations, PlanetilerConfig config, Stats stats) {
@@ -146,7 +148,7 @@ public List<VectorTile.Feature> postProcess(int zoom, List<VectorTile.Feature> l
       .collect(Collectors.groupingBy(f -> f.attrs().get(TEMP_PARTITION)))
       .values().stream()
       .map(
-        g -> g.stream().min(Comparator.comparing(i -> (Boolean) i.attrs().get(TEMP_HAS_NAME), Boolean::compare))
+        g -> g.stream().min(BY_TEMP_HAS_NAME)
       )
       .filter(Optional::isPresent)
       .map(Optional::get)