On Mon, 13 Feb 2023 19:35:52 GMT, Mandy Chung <mch...@openjdk.org> wrote:
> I overlooked in the fix for JDK-8297757 that it should have passed the > declaring class of the static fields rather than the reference class passed > to `Lookup::findStaticVarHandle`. Looks right and you've added good test coverage. test/jdk/java/lang/invoke/VarHandles/describeConstable/DescribeConstableTest.java line 53: > 51: // resolved to the one defined in the direct > superinterface of C > 52: Arguments.of(p.C.class, "stringField", String.class, > p.I.class, "I"), > 53: Arguments.of(p.C.class, "longField", long.class, > p.I.class, 10L), This looks right - when not declared in p.C, it should look next in the direct superinterfaces before looking in the super classes. ------------- Marked as reviewed by alanb (Reviewer). PR: https://git.openjdk.org/jdk/pull/12543
