RFR: 8347826: Introspector shows wrong method list after 8071693 [v4]
Phil Race
prr at openjdk.org
Thu Feb 6 18:11:26 UTC 2025
On Thu, 6 Feb 2025 14:13:50 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 fixed the test, and added the approptiate test case.
>
> Roman Marchenko has updated the pull request incrementally with one additional commit since the last revision:
>
> Update test/jdk/java/beans/Introspector/DefaultMethodBeanPropertyTest.java
>
> Co-authored-by: Aleksandr Zvegintsev <77687766+azvegint at users.noreply.github.com>
https://openjdk.org/groups/client-libs/
See "Code Reviews" where it says
"The standard requirement in the JDK project is for one reviewer to approve the fix.
The Java Client Library Group has always standardized on two approvals."
...
"The fixer therefore needs to understand the policies and wait for a second approval."
-------------
PR Comment: https://git.openjdk.org/jdk/pull/23443#issuecomment-2640628859
More information about the client-libs-dev
mailing list