On Fri, 28 Feb 2025 04:00:19 GMT, Sergey Bylokhov <s...@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
