Fixed idea warnings for class Json

javadev · Mar 25, 2024 · ef96ef2 · ef96ef2
1 parent 2779fd2
commit ef96ef2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/main/java/com/github/underscore/Json.java b/src/main/java/com/github/underscore/Json.java
@@ -407,18 +407,16 @@ private static void escape(String s, StringBuilder sb) {
                     case '\t':
                         sb.append("\\t");
                         break;
-                    case '\u20AC':
-                        sb.append('\u20AC');
+                    case '€':
+                        sb.append('€');
                         break;
                     default:
                         if (ch <= '\u001F'
                                 || ch >= '\u007F' && ch <= '\u009F'
                                 || ch >= '\u2000' && ch <= '\u20FF') {
                             String ss = Integer.toHexString(ch);
                             sb.append("\\u");
-                            for (int k = 0; k < 4 - ss.length(); k++) {
-                                sb.append("0");
-                            }
+                            sb.append("0".repeat(4 - ss.length()));
                             sb.append(ss.toUpperCase());
                         } else {
                             sb.append(ch);

diff --git a/src/test/java/com/github/underscore/LodashTest.java b/src/test/java/com/github/underscore/LodashTest.java
@@ -1563,6 +1563,9 @@ void formatJson() {
         assertEquals(
                 "{\n    \"a\": {\n    }\n}",
                 U.formatJson("{\n  \"a\": {\n  }\n}", Json.JsonStringBuilder.Step.FOUR_SPACES));
+        assertEquals(
+                "{\n   \"a\": {\n   }\n}",
+                U.formatJson("{\n  \"a\": {\n  }\n}", Json.JsonStringBuilder.Step.THREE_SPACES));
         assertEquals(
                 "{\"a\":{}}",
                 U.formatJson("{\n  \"a\": {\n  }\n}", Json.JsonStringBuilder.Step.COMPACT));