RFR: 8347826: Introspector shows wrong method list after 8071693 [v7]
Roman Marchenko
rmarchenko at openjdk.org
Thu Feb 27 11:32:00 UTC 2025
On Thu, 27 Feb 2025 11:29:36 GMT, Roman Marchenko <rmarchenko at openjdk.org> wrote:
>> Fixed `com.sun.beans.introspect.MethodInfo#MethodOrder` to make `Introspector.addMethod()` working properly when filtering methods out.
>>
>> Also, after PR discussion, added the approptiate test cases with corresponding fixes in MethodInfo.java and PropertyInfo.java.
>
> Roman Marchenko has updated the pull request incrementally with one additional commit since the last revision:
>
> Symmetric fix
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.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23443#discussion_r1973398822
More information about the client-libs-dev
mailing list