On Wed, 19 Apr 2023 22:05:33 GMT, Archie L. Cobbs <[email protected]> wrote:

>> The `Introspector` class was never updated to include `default` methods 
>> inherited from interfaces.
>> 
>> This patch attempts to fix that omission.
>
> 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 
112:

> 110:                     if ((method.getModifiers() & Modifier.ABSTRACT) == 0)
> 111:                         (list = createIfNeeded(list)).add(method);
> 112:                 }

Please always use braces even if the body has only one statement.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13544#discussion_r1172437134

Reply via email to