On Thu, 2 Jul 2026 09:47:43 GMT, ExE Boss <[email protected]> wrote:

>> src/java.base/share/classes/java/util/Objects.java line 238:
>> 
>>> 236:      * @since 28
>>> 237:      */
>>> 238:     @PreviewFeature(feature = PreviewFeature.Feature.VALUE_OBJECTS, 
>>> reflective = true)
>> 
>> Changing the requireIdentity methods to be reflective preview APIs looks 
>> okay, as is the update to their docs.  My guess is that the libraries that 
>> want to get a head start will access the preview APIs with core reflection 
>> so they won't care if the methods are just normal preview APIs.
>
> For a similar reason, `IdentityException` should probably also be made 
> reflective.

I don't think programs have a similarly strong reason to handle 
`IdentityException`, which is the only case I deem making that class valuable.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2578#discussion_r3513833844

Reply via email to