JDK 15 RFR of JDK-8225540: In core reflection note whether returned annotations are declaration or type annotations

Pavel Rappo pavel.rappo at oracle.com
Thu Apr 9 10:46:33 UTC 2020


> http://cr.openjdk.java.net/~darcy/8225540.1/

I'm not an expert in this area, but AFAIK this looks good.

A couple of nits.

1. The newly added text in the top-level doc comment for AnnotatedElement is missing some <p> tags. It looks nice in source, but it won't be like that in the output. (For better or worse, this is not Markdown.)

2. @implSpec for AnnotatedElement.isAnnotationPresent reads pretty unconventionally. There's nothing wrong with that. I simply wonder if there are any reasons why you picked that wording as opposed to the typical "The default implementation calls/returns/performs/etc. ..."? Also, there's a typo: "the the".

3. Since we are in this area, would you care to fix a couple of typos?

diff -r 193e4179def8 src/java.base/share/classes/java/lang/reflect/Modifier.java
--- a/src/java.base/share/classes/java/lang/reflect/Modifier.java       Wed Apr 08 22:58:42 2020 -0700
+++ b/src/java.base/share/classes/java/lang/reflect/Modifier.java       Thu Apr 09 11:28:32 2020 +0100
@@ -377,7 +377,7 @@
 
     /**
      * The Java source modifiers that can be applied to a method.
-     * @jls8.4.3  Method Modifiers
+     * @jls 8.4.3 Method Modifiers
      */
     private static final int METHOD_MODIFIERS =
         Modifier.PUBLIC         | Modifier.PROTECTED    | Modifier.PRIVATE |
@@ -400,9 +400,6 @@
     private static final int PARAMETER_MODIFIERS =
         Modifier.FINAL;
 
-    /**
-     *
-     */
     static final int ACCESS_MODIFIERS =
         Modifier.PUBLIC | Modifier.PROTECTED | Modifier.PRIVATE;

-Pavel



More information about the core-libs-dev mailing list