On Fri, 4 Sep 2026 18:08:52 GMT, Naoto Sato <[email protected]> wrote:

>> This PR implements [JEP 540: Simple JSON API 
>> (Incubator)](https://openjdk.org/jeps/540).
>> 
>> It adds the `jdk.incubator.json` module which provides APIs for reading and 
>> writing JSON documents as specified by [RFC 
>> 8259](https://datatracker.ietf.org/doc/html/rfc8259). This is an incubating 
>> API.
>> 
>> API documentation: 
>> https://cr.openjdk.org/~naoto/json/javadoc/api/jdk.incubator.json/module-summary.html
>> Co-authored-by: Justin Lu 
>> ([@justin-curtis-lu](https://github.com/justin-curtis-lu)), Stuart Marks 
>> ([@stuart-marks](https://github.com/stuart-marks)) 
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Naoto Sato has updated the pull request incrementally with three additional 
> commits since the last revision:
> 
>  - Merge remote-tracking branch 'jdk-sandbox/json' into 
> JDK-8381976-Implementation-for-Simple-JSON-API
>  - Replace in-line occurrences of RFC 8259 with link
>  - CSR review - JV.toString()

src/jdk.incubator.json/share/classes/jdk/incubator/json/JsonObject.java line 38:

> 36:  * The interface that represents JSON object.
> 37:  * <p>
> 38:  * A {@code JsonObject} can be produced by {@link Json#parse(String)}.

At some point, the package description was updated to have "can be done with 
Json.parse(String) or Json.parse(char[])".  Do you want to add the overload to 
JsonObject (and the other classes) too. Initially I was wondering about 
"produce" vs. "done" too but it's probably okay for now, maybe smooth out some 
of the wording in a future update.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/32282#discussion_r3947596563

Reply via email to