[lworld] RFR: 8346307: [lworld] Clarify identity vs value in Class, Objects, and document limitations of value objects [v9]
Roger Riggs
rriggs at openjdk.org
Wed Jun 25 22:05:56 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 16 commits:
- A few editorial changes
Highlight that in preview-mode, Objects can have either identity or value subclasses.
- Merge branch 'lworld' into 8346307-throws-identityexception
- Merge branch 'lworld' into 8346307-throws-identityexception
- "value interface" is a left over from an earlier design and is not allowed by the JEP 401 JLS.
An interface will never have the ACC_IDENTITY modifier, hence 'isValue()' will be true but is not significant.
Class.toGenericString() should only include the "value" prefix for record and regular classes.
Correct the test to match the toGenericString() output.
- 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
- ... and 6 more: https://git.openjdk.org/valhalla/compare/81fbc414...6cbade17
-------------
Changes: https://git.openjdk.org/valhalla/pull/1327/files
Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=1327&range=08
Stats: 266 lines in 12 files changed: 204 ins; 17 del; 45 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