On Wed, 19 Apr 2023 21:40:14 GMT, Chen Liang <li...@openjdk.org> wrote:
>> Archie L. Cobbs has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Use Set.of() to initialize IGNORABLE_INTERFACES set. > > src/java.desktop/share/classes/com/sun/beans/introspect/MethodInfo.java line > 46: > >> 44: final class MethodInfo { >> 45: >> 46: static final HashSet<Class<?>> IGNORABLE_INTERFACES = new >> HashSet<>(6); > > Suggestion: > > static final HashSet<Class<?>> IGNORABLE_INTERFACES = > HashSet.newHashSet(6); > > Or even better, use Set.of instead. Thanks, much cleaner. Fixed in e37a1467a57. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13544#discussion_r1171894172