[lworld] RFR: 8346307: [lworld] Clarify identity vs value in Class, Objects, and document limitations of value objects [v8]

Chen Liang liach at openjdk.org
Fri May 30 01:36:08 UTC 2025


On Thu, 29 May 2025 19:48:45 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> Add APINote  and javadoc for IdentityException where it will be useful to know that identity or value objects are treated differently. 
>> Simplified WeakHashMap javadoc updates for IdentityException. 
>> Added note to System.identityHashCode to include value objects.
>> Added to class javadoc for IdentityHashMap for value objects.
>
> Roger Riggs has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 12 commits:
> 
>  - Merge branch 'lworld' into 8346307-throws-identityexception
>  - Merge branch 'lworld' into 8346307-throws-identityexception
>  - Merge
>  - Merge branch 'lworld' into 8346307-throws-identityexception
>  - Merge branch 'lworld' into 8346307-throws-identityexception
>  - Merge branch 'lworld' into 8346307-throws-identityexception
>  - Merge
>  - Expand javadoc reference to Reference
>  - Merge remote-tracking branch 'refs/remotes/origin/8346307-throws-identityexception' into 8346307-throws-identityexception
>  - Merge branch 'openjdk:lworld' into 8346307-throws-identityexception
>  - ... and 2 more: https://git.openjdk.org/valhalla/compare/be146831...b6cd55b5

test/jdk/java/lang/Class/GenericStringTest.java line 60:

> 58:                                         "public abstract class java.lang.Enum<E extends java.lang.Enum<E>>"),
> 59:                    new PlatformTestCase(java.util.Map.class,
> 60:                                         "public abstract value interface java.util.Map<K,V>"),

Hmm, isn't value redundant on interfaces?

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1327#discussion_r2114990571


More information about the valhalla-dev mailing list