src/java.base/share/classes/java/lang/reflect/Modifier.java

@@ -23,27 +23,30 @@
   * questions.
   */
  
  package java.lang.reflect;
  
+ import jdk.internal.javac.PreviewFeature;
+ 
  import java.util.StringJoiner;
  
  /**
   * The Modifier class provides {@code static} methods and
-  * constants to decode class and member access modifiers.  The sets of
-  * modifiers are represented as integers with distinct bit positions
+  * constants to decode class and member access modifiers.
+  * The {@link AccessFlag} class should be used instead of this class.
+  * The sets of modifiers are represented as integers with non-distinct bit positions
   * representing different modifiers.  The values for the constants
   * representing the modifiers are taken from the tables in sections
   * {@jvms 4.1}, {@jvms 4.4}, {@jvms 4.5}, and {@jvms 4.7} of
   * <cite>The Java Virtual Machine Specification</cite>.
   *
   * @apiNote
   * Not all modifiers that are syntactic Java language modifiers are
   * represented in this class, only those modifiers that <em>also</em>
   * have a corresponding JVM {@linkplain AccessFlag access flag} are
-  * included. In particular the {@code default} method modifier (JLS
-  * {@jls 9.4.3}) and the {@code sealed} and {@code non-sealed} class
+  * included. In particular, the {@code default} method modifier (JLS
+  * {@jls 9.4.3}) and the {@code value}, {@code sealed} and {@code non-sealed} class
   * (JLS {@jls 8.1.1.2}) and interface (JLS {@jls 9.1.1.4}) modifiers
   * are <em>not</em> represented in this class.
   *
   * @see Class#getModifiers()
   * @see Member#getModifiers()

@@ -121,18 +124,39 @@
  
      /**
       * Return {@code true} if the integer argument includes the
       * {@code synchronized} modifier, {@code false} otherwise.
       *
+      * @apiNote {@code isSynchronized} should only be called with the modifiers
+      * of a {@linkplain Method#getModifiers() method}.
+      *
       * @param   mod a set of modifiers
       * @return {@code true} if {@code mod} includes the
       * {@code synchronized} modifier; {@code false} otherwise.
       */
      public static boolean isSynchronized(int mod) {
          return (mod & SYNCHRONIZED) != 0;
      }
  
+     /**
+      * Return {@code true} if the integer argument includes the
+      * {@code identity} modifier, {@code false} otherwise.
+      *
+      * @apiNote {@code isIdentity} should only be called with the modifiers
+      * of a {@linkplain Class#getModifiers() class}.
+      *
+      * @param   mod a set of modifiers
+      * @return {@code true} if {@code mod} includes the
+      * {@code identity} modifier; {@code false} otherwise.
+      *
+      * @since Valhalla
+      */
+     @PreviewFeature(feature = PreviewFeature.Feature.VALUE_OBJECTS, reflective=true)
+     public static boolean isIdentity(int mod) {
+         return (mod & IDENTITY) != 0;
+     }
+ 
      /**
       * Return {@code true} if the integer argument includes the
       * {@code volatile} modifier, {@code false} otherwise.
       *
       * @param   mod a set of modifiers

@@ -205,11 +229,11 @@
  
      /**
       * Return a string describing the access modifier flags in
       * the specified modifier. For example:
       * <blockquote><pre>
-      *    public final synchronized strictfp
+      *    public final synchronized
       * </pre></blockquote>
       * The modifier names are returned in an order consistent with the
       * suggested modifier orderings given in sections 8.1.1, 8.3.1, 8.4.3, 8.8.3, and 9.1.1 of
       * <cite>The Java Language Specification</cite>.
       * The full modifier ordering used by this method is:

@@ -317,10 +341,19 @@
       * modifier.
       * @see AccessFlag#SYNCHRONIZED
       */
      public static final int SYNCHRONIZED     = 0x00000020;
  
+     /**
+      * The {@code int} value representing the {@code ACC_IDENTITY}
+      * modifier.
+      *
+      * @since Valhalla
+      */
+     @PreviewFeature(feature = PreviewFeature.Feature.VALUE_OBJECTS, reflective=true)
+     public static final int IDENTITY         = 0x00000020;
+ 
      /**
       * The {@code int} value representing the {@code volatile}
       * modifier.
       * @see AccessFlag#VOLATILE
       */

@@ -355,11 +388,11 @@
      public static final int ABSTRACT         = 0x00000400;
  
      /**
       * The {@code int} value representing the {@code strictfp}
       * modifier.
-      * @see AccessFlag#STRICT
+      * @see AccessFlag#STRICT and AccessFlag#STRICT_FIELD
       */
      public static final int STRICT           = 0x00000800;
  
      // Bits not (yet) exposed in the public API either because they
      // have different meanings for fields and methods and there is no

@@ -394,11 +427,11 @@
       * @jls 8.1.1 Class Modifiers
       */
      private static final int CLASS_MODIFIERS =
          Modifier.PUBLIC         | Modifier.PROTECTED    | Modifier.PRIVATE |
          Modifier.ABSTRACT       | Modifier.STATIC       | Modifier.FINAL   |
-         Modifier.STRICT;
+         Modifier.IDENTITY       | Modifier.STRICT;
  
      /**
       * The Java source modifiers that can be applied to an interface.
       * @jls 9.1.1 Interface Modifiers
       */

@@ -428,11 +461,11 @@
       * @jls 8.3.1 Field Modifiers
       */
      private static final int FIELD_MODIFIERS =
          Modifier.PUBLIC         | Modifier.PROTECTED    | Modifier.PRIVATE |
          Modifier.STATIC         | Modifier.FINAL        | Modifier.TRANSIENT |
-         Modifier.VOLATILE;
+         Modifier.VOLATILE       | Modifier.STRICT;
  
      /**
       * The Java source modifiers that can be applied to a method or constructor parameter.
       * @jls 8.4.1 Formal Parameters
       */