Integrated: 8280132: Incorrect comparator com.sun.beans.introspect.MethodInfo.MethodOrder

Sergey Bylokhov serb at openjdk.java.net
Wed Feb 9 01:30:10 UTC 2022


On Sun, 23 Jan 2022 07:41:39 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

> In jdk 9 we started to sort the list of methods for each class for two reasons:
>  1. We had a number of bugs which state that our JavaBeans randomly does not work, examples: JDK-6807471[1] , JDK-6788525[2], the reason was that the order of methods from Class.getMethods() is not specified.
>  2. We tried to sort methods so the more specific returns types come first, this was done because our logic for selecting the correct method did not work properly. 
> 
> The second issue above was fixed by the separate change [JDK-8196373](https://bugs.openjdk.java.net/browse/JDK-8196373) so now we only need to sort the list of methods in any order and do not care about return types.

This pull request has now been integrated.

Changeset: 2f46af05
Author:    Sergey Bylokhov <serb at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/2f46af05ce2d43e19e0095680eb3a52fd904c774
Stats:     1542 lines in 2 files changed: 1513 ins; 14 del; 15 mod

8280132: Incorrect comparator com.sun.beans.introspect.MethodInfo.MethodOrder

Reviewed-by: prr

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

PR: https://git.openjdk.java.net/jdk/pull/7190



More information about the client-libs-dev mailing list