RFR: 8071693: Introspector ignores default interface methods [v2]

Archie L. Cobbs duke at openjdk.org
Wed Apr 19 22:05:36 UTC 2023


On Wed, 19 Apr 2023 21:40:14 GMT, Chen Liang <liach at 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



More information about the client-libs-dev mailing list