On Wed, 19 Aug 2026 13:58:44 GMT, Per Minborg <[email protected]> wrote:
> Would it not be better to move the implementation classes to `jdk.internal`
> just like the Vector API does?
@minborg `java.base` cannot depend on an incubating module.
> src/jdk.incubator.json/share/classes/jdk/incubator/json/impl/JsonObjectImpl.java
> line 66:
>
>> 64: Objects.requireNonNull(name);
>> 65: return switch (theMembers.get(name)) {
>> 66: case JsonValue jv -> jv;
>
> An enhanced switch with pattern matching is convenient and easy to read but
> adds to startup time. Did we do this trade-off analysis?
I don't think avoidance of lambda or startup time is a focus of this library.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/32282#issuecomment-5343327059
PR Review Comment: https://git.openjdk.org/jdk/pull/32282#discussion_r3813734949
