On Tue, 8 Aug 2023 11:54:42 GMT, Pavel Rappo <pra...@openjdk.org> wrote:

>> src/java.security.jgss/share/classes/sun/security/krb5/internal/HostAddress.java
>>  line 105:
>> 
>>> 103:             return false;
>>> 104:         if (address != null && h.address != null) {
>>> 105:             return Arrays.equals(address, h.address);
>> 
>> Can't this block be combined into:
>> return` addrType == h.addrType && Arrays.equals(address, h.address); `?
>
> Have a look at the end result (commits e71fc42a173 and d9316270976), is it 
> better?

Oops: forgot to commit and push 608258e.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14738#discussion_r1287356149

Reply via email to