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

Roger Riggs rriggs at openjdk.org
Tue Mar 25 18:51:02 UTC 2025


> 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 eight commits:

 - 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
 - 8346307: [lworld] Document WeakHashMap and other apis that may throw IdentityException
   Add APINote  and javadoc for IdentityException where it will be useful to know.
   Simplified WeakHashMap javadoc updates for IdentityException.
   Added note to System.identityHashCode to include value objects.
 - 8346307: [lworld] Document WeakHashMap and other apis that may throw IdentityException
   Add APINote and javadoc for IdentityException where it will be useful to know.
   Updated Objects.hasIdentity to return true for non-null reference to identity class; else false
   Added Objects.isValueObject to return true for non-null reference to value class; else false
   Updated tests for value objects.

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

Changes: https://git.openjdk.org/valhalla/pull/1327/files
  Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=1327&range=05
  Stats: 250 lines in 11 files changed: 193 ins; 14 del; 43 mod
  Patch: https://git.openjdk.org/valhalla/pull/1327.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/1327/head:pull/1327

PR: https://git.openjdk.org/valhalla/pull/1327


More information about the valhalla-dev mailing list