Changeset 17509 in josm for trunk/src/org/openstreetmap/josm/data

Timestamp:

2021-02-21T16:10:35+01:00 (3 years ago)

Author:

wiktorn

Message:

Checkstyle fixes

Location:

trunk/src/org/openstreetmap/josm/data

Files:

• cache/JCSCachedTileLoaderJob.java (modified) (1 diff)
• imagery/TMSCachedTileLoaderJob.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/data/cache/JCSCachedTileLoaderJob.java

@@ -188 +188 @@
      * Simple implementation. All errors should be cached as empty. Though some JDK (JDK8 on Windows for example)
      * doesn't return 4xx error codes, instead they do throw an FileNotFoundException or IOException
-     * @param responseCode
-     * @param headerFields
+     * @param headerFields headers sent by server
+     * @param responseCode http status code
      *
      * @return true if we should put empty object into cache, regardless of what remote resource has returned

trunk/src/org/openstreetmap/josm/data/imagery/TMSCachedTileLoaderJob.java

@@ -136 +136 @@
         }
         if (isNotImage(headers, statusCode)) {
-            String message = detectErrorMessage( new String(content, StandardCharsets.UTF_8));
+            String message = detectErrorMessage(new String(content, StandardCharsets.UTF_8));
             if (message != null && !message.isEmpty()) {
                 tile.setError(message);
@@ -148 +148 @@
         if (statusCode == 200 && headers.containsKey("Content-Type") && !headers.get("Content-Type").isEmpty()) {
             String contentType = headers.get("Content-Type").stream().findAny().get();
-            if (contentType !=null && !contentType.startsWith("image")) {
+            if (contentType != null && !contentType.startsWith("image")) {
                 Logging.warn("Image not returned for tile: " + url + " content type was: " + contentType);
                 // not an image - do not store response in cache, so next time it will be queried again from the server