[java] Fixing minor issues reported by IDEA code linter

Author: barancev

java/client/src/org/openqa/selenium/MutableCapabilities.java

@@ -202,24 +202,24 @@ private String abbreviate(Map<Object, String> seen, Object stringify) {
     if (stringify.getClass().isArray()) {
       value.append("[");
       value.append(
-          Stream.of((Object[]) stringify)
-              .map(item -> abbreviate(seen, item))
-              .collect(Collectors.joining(", ")));
+        Stream.of((Object[]) stringify)
+          .map(item -> abbreviate(seen, item))
+          .collect(Collectors.joining(", ")));
       value.append("]");
     } else if (stringify instanceof Collection) {
       value.append("[");
       value.append(
-          ((Collection<?>) stringify).stream()
-              .map(item -> abbreviate(seen, item))
-              .collect(Collectors.joining(", ")));
+        ((Collection<?>) stringify).stream()
+          .map(item -> abbreviate(seen, item))
+          .collect(Collectors.joining(", ")));
       value.append("]");
     } else if (stringify instanceof Map) {
       value.append("{");
       value.append(
-          ((Map<?, ?>) stringify).entrySet().stream()
-              .sorted(Comparator.comparing(entry -> String.valueOf(entry.getKey())))
-              .map(entry -> String.valueOf(entry.getKey()) + ": " + abbreviate(seen, entry.getValue()))
-              .collect(Collectors.joining(", ")));
+        ((Map<?, ?>) stringify).entrySet().stream()
+          .sorted(Comparator.comparing(entry -> String.valueOf(entry.getKey())))
+          .map(entry -> String.format("%s: %s", entry.getKey(), abbreviate(seen, entry.getValue())))
+          .collect(Collectors.joining(", ")));
       value.append("}");
     } else {
       String s = String.valueOf(stringify);

java/client/src/org/openqa/selenium/json/JsonType.java

@@ -28,5 +28,4 @@ public enum JsonType {
   END_COLLECTION,
   STRING,
   END
-  ;
 }

java/client/src/org/openqa/selenium/json/PropertySetting.java

@@ -20,5 +20,4 @@
 public enum PropertySetting {
   BY_NAME,
   BY_FIELD
-  ;
 }

java/client/src/org/openqa/selenium/net/UrlChecker.java

@@ -21,7 +21,6 @@
 import java.net.HttpURLConnection;
 import java.net.URL;
 import java.util.Arrays;
-import java.util.concurrent.Callable;
 import java.util.concurrent.ExecutionException;
 import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;

java/client/src/org/openqa/selenium/remote/codec/w3c/W3CHttpCommandCodec.java

@@ -435,7 +435,7 @@ private Map<String, String> asElement(Object id) {
   private String cssEscape(String using) {
     using = using.replaceAll("([\\s'\"\\\\#.:;,!?+<>=~*^$|%&@`{}\\-\\/\\[\\]\\(\\)])", "\\\\$1");
     if (using.length() > 0 && Character.isDigit(using.charAt(0))) {
-      using = "\\" + Integer.toString(30 + Integer.parseInt(using.substring(0,1))) + " " + using.substring(1);
+      using = "\\" + (30 + Integer.parseInt(using.substring(0,1))) + " " + using.substring(1);
     }
     return using;
   }

java/client/src/org/openqa/selenium/remote/tracing/Span.java

@@ -46,7 +46,7 @@ enum Kind {
     // The nice name is the name expected in an OT trace.
     private final String niceName;
 
-    private Kind(String niceName) {
+    Kind(String niceName) {
       this.niceName = niceName;
     }