On Mon, 17 Aug 2026 08:59:55 GMT, Alan Bateman <[email protected]> wrote:

>> 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
>
> src/java.base/share/classes/module-info.java line 149:
> 
>> 147:         java.security.sasl;
>> 148:     exports jdk.internal to
>> 149:         jdk.incubator.json,
> 
> I assume this qualified export is not needed, maybe left over from 
> experiments with `@ValueBased` or maybe copies from the setup for the vector 
> API?

Right. I will remove it from the module info

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

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

Reply via email to