On Wed, 19 Aug 2026 14:01:12 GMT, Per Minborg <[email protected]> wrote:

>> 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`?

I don't know how important performance is for first incuabtor but just to say 
that the incubating module is mapped to the app class loader so it cannot use 
these annotations.

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

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

Reply via email to