> 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. > > --------- > `getMethodDescriptors()` returns descriptors of public methods of a class and > its parent classes, including default and static methods defined in > interfaces. The result doesn't include methods which were declared and not > implemented, bridge methods, or methods which were overriden in subclasses. > > When a subclass "overrides" a static method from a parent class > `getMethodDescriptors()` behaves as follows: > > - In case of a clash with a class method, the class version will take > precedence rather than interface's one. > - If the same method appears in multiple classes/interfaces within the same > hierarchy, the version from the most specific sub-interface will be selected. > - If the method appears in classes/interfaces from different hierarchies, the > result is undefined, either version can be returned. > > `getPropertyDescriptors()` returns descriptors of methods which were > identified as getters or setters. As there can be the only method > getter/setter per property, the following rules are applied when choosing a > getter/setter: > > * Getters/setters for the same property defined (not necessarily overriden) > in subclasses have higher precedence. > * If there are getters/setters for the same property defined in the same > class and argument types are assignable one to another, the method with the > Least Common Supertype has the lower priority. If argument types are not > assignable, the result is undefined (any method will be chosen). > * Gettters/setters declared and not implemented are not considered.
Roman Marchenko has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 18 additional commits since the last revision: - Merge branch 'openjdk:master' into jdk-8347826 - Restored the original order processing - Fixing review comments - Implementing new requirements - Merge branch 'master' into jdk-8347826 - Fixing review comments - Fixing review comments 2 - Symmetric fix - minor fix - minor fix - ... and 8 more: https://git.openjdk.org/jdk/compare/b63c9c9a...779194f1 ------------- Changes: - all: https://git.openjdk.org/jdk/pull/23443/files - new: https://git.openjdk.org/jdk/pull/23443/files/431705a0..779194f1 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=23443&range=13 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=23443&range=12-13 Stats: 436893 lines in 5240 files changed: 174030 ins; 237687 del; 25176 mod Patch: https://git.openjdk.org/jdk/pull/23443.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/23443/head:pull/23443 PR: https://git.openjdk.org/jdk/pull/23443
