On Wed, 19 Aug 2026 14:03:17 GMT, Per Minborg <[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/JsonBoolean.java line
> 42:
>
>> 40: * @since 28
>> 41: */
>> 42: public non-sealed interface JsonBoolean extends JsonValue {
>
> What is the rationale for leaving these interfaces `non-sealed`? While this
> allows anyone to implement `JsonValue` classes, it expands the API commitment
> significantly.
This is probably something that should be in the JEP as there will be "why"
questions. As I recall, there was interest in areas such as database to
implementations that were optimized representations. I see it as something that
can be evaluated while the API is incubating.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/32282#discussion_r3814204262
