On Thu, 20 Aug 2026 08:18:05 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 %04X format for invalid hex digit error message
>> - Update outdated comment in toPath
>> - Renamed `JsonValueImpl` to `JsonValueSupport`
>
> src/jdk.incubator.json/share/classes/jdk/incubator/json/impl/JsonParser.java
> line 138:
>
>> 136: if (members.containsKey(name)) {
>> 137: throw failure(nameStart, nameLine, nameLineStart,
>> 138: "Duplicate member name: \"%s\" was already
>> parsed".formatted(name), startO, true);
>
> This is another case where a name containing a LF or other interesting
> characters makes for a confusing problematic exception message.
Right, we can escape the name string here on output.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/32282#discussion_r3823533546
