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

Sergey Bylokhov serb at openjdk.org
Wed Mar 26 20:03:21 UTC 2025


On Wed, 26 Mar 2025 19:58:22 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

>> BTW please merge the master branch, the current branch is way too outdated.
>
>> Redarding properties, I think the implementation is OK. The only question in this case is should `getMethodDescriptors()` contain the static method (`public static int DefaultMethodBeanPropertyTest$A4.getStaticValue()`) or not?
> 
> It looks like for classes it was implemented as follows:
>  - getPropertyDescriptors() checks only instance methods getXXX()/setXXX()
>  - getMethodDescriptors() returns all methods of the class, including static ones, but static methods are not considered properties
> 
> It would be good to cover this by the test

Also, in the code above, it is not necessary to filter out private methods since iface.getMethods() returns only public

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

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


More information about the client-libs-dev mailing list