On Sat, 17 Jun 2023 00:35:22 GMT, Mandy Chung <mch...@openjdk.org> wrote:
>> This PR clarifies the spec of the 3-arg Class::forName regarding the format >> of the name for an array type which is of the form: one or more of "[" + >> binary name of the element type + ";'. > > Mandy Chung has updated the pull request incrementally with one additional > commit since the last revision: > > missing 'L' for the array class name src/java.base/share/classes/java/lang/Class.java line 395: > 393: * attached thread), the system class loader is used. > 394: * > 395: * @param className the <a href="#binary-name">binary name</a> > of the class Where do these links lead? I don't find any section with this ID in the `Class` Javadoc. Should they lead to `ClassLoader`? That Javadoc has such a section. src/java.base/share/classes/java/lang/Class.java line 427: > 425: * Returns the {@code Class} object associated with the class or > 426: * interface with the given string name, using the given class > loader. > 427: * Given the {@linkplain ##binary-name binary name} for a class or > interface, Is double hash a mistake? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/14528#discussion_r1233359948 PR Review Comment: https://git.openjdk.org/jdk/pull/14528#discussion_r1233360226