On Thu, 8 Jun 2023 07:22:16 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:
>> Prasanta Sadhukhan has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Address hashcode review comment > > src/java.desktop/share/classes/javax/swing/text/html/CSS.java line 2684: > >> 2682: if (units != null) { >> 2683: hashCode |= units.hashCode(); >> 2684: } > > In previous cases, you used `Objects.hashCode(units)`, I presume, to avoid > explicit null-check and branches. Wouldn't it work here too? Yes, we can do it ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13405#discussion_r1222820846