Changeset 18011 in josm for trunk/src/org

Timestamp:

2021-07-13T22:49:09+02:00 (3 years ago)

Author:

Don-vip

Message:

see #17177 - add debug log on saving, wait for saving in testMapillaryStyle, code style

Location:

trunk/src/org/openstreetmap/josm

Files:

• data/imagery/vectortile/mapbox/style/MapboxVectorStyle.java (modified) (10 diffs)
• gui/mappaint/mapcss/MapCSSStyleSource.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/data/imagery/vectortile/mapbox/style/MapboxVectorStyle.java

@@ -17 +17 @@
 import java.util.List;
 import java.util.Map;
+import java.util.Map.Entry;
 import java.util.Objects;
 import java.util.Optional;
@@ -24 +25 @@
 import javax.imageio.ImageIO;
 import javax.json.Json;
-import javax.json.JsonArray;
 import javax.json.JsonObject;
 import javax.json.JsonReader;
-import javax.json.JsonStructure;
 import javax.json.JsonValue;
 
@@ -60 +59 @@
                     BufferedReader reader = style.getContentReader();
                     JsonReader jsonReader = Json.createReader(reader)) {
-                JsonStructure structure = jsonReader.read();
-                return new MapboxVectorStyle(structure.asJsonObject());
+                return new MapboxVectorStyle(jsonReader.read().asJsonObject());
             } catch (IOException e) {
                 Logging.error(e);
@@ -100 +98 @@
             final List<Source> sourceList;
             if (jsonObject.containsKey("sources") && jsonObject.get("sources").getValueType() == JsonValue.ValueType.OBJECT) {
-                final JsonObject sourceObj = jsonObject.getJsonObject("sources");
-                sourceList = sourceObj.entrySet().stream().filter(entry -> entry.getValue().getValueType() == JsonValue.ValueType.OBJECT)
+                sourceList = jsonObject.getJsonObject("sources").entrySet().stream()
+                  .filter(entry -> entry.getValue().getValueType() == JsonValue.ValueType.OBJECT)
                   .map(entry -> {
                       try {
@@ -119 +117 @@
             final List<Layers> layers;
             if (jsonObject.containsKey("layers") && jsonObject.get("layers").getValueType() == JsonValue.ValueType.ARRAY) {
-                JsonArray lArray = jsonObject.getJsonArray("layers");
-                layers = lArray.stream().filter(JsonObject.class::isInstance).map(JsonObject.class::cast).map(obj -> new Layers(id, obj))
+                layers = jsonObject.getJsonArray("layers").stream()
+                  .filter(JsonObject.class::isInstance).map(JsonObject.class::cast).map(obj -> new Layers(id, obj))
                   .collect(Collectors.toList());
             } else {
@@ -130 +128 @@
             // Abuse HashMap null (null == default)
             this.sources = new LinkedHashMap<>();
-            for (Map.Entry<Optional<Source>, List<Layers>> entry : sourceLayer.entrySet()) {
+            for (Entry<Optional<Source>, List<Layers>> entry : sourceLayer.entrySet()) {
                 final Source source = entry.getKey().orElse(null);
                 final String data = entry.getValue().stream().map(Layers::toString).collect(Collectors.joining());
@@ -159 +157 @@
         // HiDPI images first -- if this succeeds, don't bother with the lower resolution (JOSM has no method to switch)
         try (CachedFile spriteJson = new CachedFile(this.spriteUrl + "@2x.json");
-          CachedFile spritePng = new CachedFile(this.spriteUrl + "@2x.png")) {
+             CachedFile spritePng = new CachedFile(this.spriteUrl + "@2x.png")) {
             if (parseSprites(spriteJson, spritePng)) {
                 return;
@@ -165 +163 @@
         }
         try (CachedFile spriteJson = new CachedFile(this.spriteUrl + ".json");
-        CachedFile spritePng = new CachedFile(this.spriteUrl + ".png")) {
+             CachedFile spritePng = new CachedFile(this.spriteUrl + ".png")) {
             parseSprites(spriteJson, spritePng);
         }
@@ -193 +191 @@
             return false;
         }
-        for (Map.Entry<String, JsonValue> entry : spriteObject.entrySet()) {
+        for (Entry<String, JsonValue> entry : spriteObject.entrySet()) {
             final JsonObject info = entry.getValue().asJsonObject();
             int width = info.getInt("width");
@@ -218 +216 @@
         }
         final File toSave = new File(location, name);
+        Logging.debug("Saving {0} to {1}...", object.getClass().getSimpleName(), toSave);
         try {
             if (object instanceof BufferedImage) {

trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSStyleSource.java

@@ -95 +95 @@
     static final Set<String> SUPPORTED_KEYS = new HashSet<>();
     static {
-        Field[] declaredFields = StyleKeys.class.getDeclaredFields();
-        for (Field f : declaredFields) {
+        for (Field f : StyleKeys.class.getDeclaredFields()) {
             try {
                 SUPPORTED_KEYS.add((String) f.get(null));