On Thu, 22 Oct 2026 13:34:20 GMT, Jan Lahoda <[email protected]> wrote:
>> Vicente Romero has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review comment > > test/jdk/java/lang/reflect/valhalla/ValueClassesReflectionTest.java line 103: > >> 101: assertTrue(pair.cls.isValue()); >> 102: Field[] fields = pair.cls.getDeclaredFields(); >> 103: assertTrue(fields.length == pair.numberOfFields); > > Nit: the declaration of `numberOfFields` field in the top-level class can > probably be dropped/removed right? right, I forgot to remove it, thanks! ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/33507#discussion_r4349104800
