On Fri, 14 Aug 2026 18:19:46 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)) >> >> --------- >> - [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 with a new target base due to a merge > or a rebase. The pull request now contains 726 commits: > > - Merge remote-tracking branch 'jdk-sandbox/json' into > JDK-8381976-Implementation-for-Simple-JSON-API > - Address comment regarding asDouble wording in class spec > - Reflects Alan's comments > - Merge branch 'master' into JDK-8381976-Implementation-for-Simple-JSON-API > - Merge remote-tracking branch 'jdk-sandbox/json' into > JDK-8381976-Implementation-for-Simple-JSON-API > - Removed unused imports > - Removed unnecessary qualifiers > - Merge remote-tracking branch 'jdk-sandbox/json' into > JDK-8381976-Implementation-for-Simple-JSON-API > - Merge branch 'toDisplayString' into json > - wording > - ... and 716 more: https://git.openjdk.org/jdk/compare/16bf2730...76272098 test/jdk/jdk/incubator/json/TestOtherImpl.java line 29: > 27: * @test > 28: * @bug 8381976 > 29: * @summary Tests for other implementations that extend JsonValue > sub-interfaces For now at least, JsonString/JsonNumber/JsonObject/JsonArray/JsonBoolean/JsonNull are non-sealed, but the test for this "extension point" only tests two custom-JsonString methods. Are there other generate tests that could be added here? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/32282#discussion_r3795168234
