RFR: 8347826: Introspector shows wrong method list after 8071693 [v7]

[Roman Marchenko]

On Fri, 28 Feb 2025 04:00:19 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

>> src/java.desktop/share/classes/com/sun/beans/introspect/PropertyInfo.java line 98:
>> 
>>> 96:                     writeType = info.type;
>>> 97:                 } else if (writeType.isAssignableFrom(info.type)) {
>>> 98:                     if ((this.write == null) || (!info.method.isDefault() && this.write.type.isAssignableFrom(info.type))) {
>> 
>> I cannot reproduce this because of the sorting, however it's reproduced without sorting.
>> Added this to make the fix symmetric between read/write properties.
>
> Do we always want to drop default methods here, or should we accept them if the "current method" is also a default method(or null)?

You're right, this is a good question.
Changed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23443#discussion_r1975148641