On Thu, 8 May 2025 08:47:50 GMT, Maurizio Cimadamore <mcimadam...@openjdk.org>
wrote:
>> Per Minborg has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Remove class
>
> src/java.base/share/classes/jdk/internal/foreign/CaptureStateUtil.java line
> 110:
>
>> 108:
>> 109: @Override
>> 110: public int hashCode() {
>
> isn't the default toString/hashCode/equals for records good enough? If these
> overload are added for bootstrapping, please say so in a comment
Added for bootstrapping but I think this could be addressed in another way
(e.g. helloClass once this class is used).
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25043#discussion_r2079258413
