On Thu, 2 Jul 2026 16:18:01 GMT, ExE Boss <[email protected]> wrote: >> Chen Liang 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 eight additional >> commits since the last revision: >> >> - Alan reviews >> - Merge branch 'lworld' of https://github.com/openjdk/valhalla into >> doc/lw-reflect-preview-api-0 >> - Merge branch 'lworld' of https://github.com/openjdk/valhalla into >> doc/lw-reflect-preview-api-0 >> - Merge branch 'lworld' of https://github.com/openjdk/valhalla into >> doc/lw-reflect-preview-api-0 >> - Merge branch 'lworld' of https://github.com/openjdk/valhalla into >> doc/lw-reflect-preview-api-0 >> - Reword recommendations >> - Make requireIdentity reflective >> - Reflective preview API behavior out of preview > > src/java.base/share/classes/java/lang/Class.java line 1394: > >> 1392: * represents either a class whose {@code class} file >> does not >> 1393: * depend on preview features or an array type, its >> {@code identity} >> 1394: * modifier is always true. > > Suggestion: > > * represents either a class whose {@code class} file does not > * depend on preview features or represents an array type, > * its {@code identity} modifier is always true.
"represents either" need to become "either represents". ------------- PR Review Comment: https://git.openjdk.org/valhalla/pull/2578#discussion_r3520090670
