On Thu, 15 Oct 2026 20:47:45 GMT, Fairoz Matte <[email protected]> wrote:

>> src/java.management/share/classes/javax/management/ImmutableDescriptor.java 
>> line 377:
>> 
>>> 375:     public int hashCode() {
>>> 376:         int hash = hashCode;
>>> 377:         if (hash == -1) {
>> 
>> The changes to all 3 looks good, I'm just wondering why -1 is used as the 
>> "not set" rather than 0.
>
> Thanks @AlanBateman and @kevinjwalls for looking into this.

Hi Alan, I prefer to keep the value as it is, -1 instead of 0. Do let me know, 
if that is fine!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/33368#discussion_r4298625048

Reply via email to