On Wed, 19 Aug 2026 14:00:21 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/impl/JsonArrayImpl.java
> line 38:
>
>> 36: * JsonArray implementation class
>> 37: */
>> 38: public final class JsonArrayImpl implements JsonArray, JsonValueImpl {
>
> Could we consider using records for the implementation classes? The
> components of a record are trusted by the VM.
Alternatively, could we consider `@TrustFinalFields`?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/32282#discussion_r3813654757
