On Tue, 18 Aug 2026 19:23:47 GMT, Alan Bateman <[email protected]> wrote:
>> Naoto Sato has updated the pull request incrementally with four additional >> commits since the last revision: >> >> - Merge remote-tracking branch 'jdk-sandbox/json' into >> JDK-8381976-Implementation-for-Simple-JSON-API >> - Use \u instead of 0x for code point prefix >> - Get rid of StringBuilder.insert(0, ... in path building >> - Reflects reviews > > src/jdk.incubator.json/share/classes/jdk/incubator/json/impl/JsonParser.java > line 470: > >> 468: "Unexpected value. Expected a JSON Object, Array, String, >> Number, Boolean, or Null"; >> 469: private static final String UNRECOGNIZED_ESCAPE_SEQUENCE = >> 470: "Unrecognized escape sequence: \"\\\\u%04X\""; > > The change from %c to u%04X is good but I think you've got an extra slash now. The extra backslash is actually the prepending escaping backslash, as this represents an (incorrect) escape sequence. E.g, `\a` is printed as `\\u0061` ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/32282#discussion_r3807481357
