RFR: JDK-8308913: Update core reflection for JEP 445 (preview) [v4]

Alan Bateman alanb at openjdk.org
Mon Jun 5 10:28:10 UTC 2023


On Fri, 2 Jun 2023 04:48:35 GMT, Joe Darcy <darcy at openjdk.org> wrote:

>> Explain in java.lang.Class how unnamed classes are modeled in core reflection.
>
> Joe Darcy has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update reflective support.

Will this method be tested by the changes in pull/13689?

src/java.base/share/classes/java/lang/Class.java line 29:

> 27: 
> 28: import jdk.internal.javac.PreviewFeature;
> 29: import jdk.internal.misc.PreviewFeatures;

Can you move this so that it's with the other imports of jdk.internal classes?

src/java.base/share/classes/java/lang/Class.java line 194:

> 192:  * <h2><a id=unnamedClasses>Unnamed Classes</a></h2>
> 193:  *
> 194:  * A {@code class} file representing an {@linkplain #isUnnamedClass unnamed class} is generated by

Are you going to reflow this paragraph before you are done, only wondering as there is one line sticking out compared the rest of the paragraph.

src/java.base/share/classes/java/lang/Class.java line 216:

> 214:  * {@linkplain #getSimpleName
> 215:  * simple name} of such an unnamed class is the empty string and the
> 216:  * {@linkplain #getCanonicalName canonical name} is {@code null}.

This sentence is quite long. You might to consider splitting it up, or at minimally put a comma after "empty string" to allow the reader draw breath.

src/java.base/share/classes/java/lang/Class.java line 1881:

> 1879:      * @since 21
> 1880:      */
> 1881:     @PreviewFeature(feature=PreviewFeature.Feature.UNNAMED_CLASSES,

Is there a JLS section to reference?

-------------

PR Review: https://git.openjdk.org/jdk/pull/14165#pullrequestreview-1462196037
PR Review Comment: https://git.openjdk.org/jdk/pull/14165#discussion_r1217845247
PR Review Comment: https://git.openjdk.org/jdk/pull/14165#discussion_r1217846409
PR Review Comment: https://git.openjdk.org/jdk/pull/14165#discussion_r1217854569
PR Review Comment: https://git.openjdk.org/jdk/pull/14165#discussion_r1217848157


More information about the core-libs-dev mailing list