Minor cleanup, preview and since

Reviewed-by: vromero

Author: liach

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

@@ -147,8 +147,9 @@ public static Object newInstance(Class<?> componentType, int... dimensions)
      * @throws ArrayStoreException if a component of {@code sourceArray} cannot
      *         be stored into the new array because of a type mismatch.
      * @throws NegativeArraySizeException if {@code length < 0}
+     * @since Valhalla
      */
-    @PreviewFeature(feature = Feature.VALUE_OBJECTS)
+    @PreviewFeature(feature = Feature.NULL_RESTRICTION)
     public static Object newInstance(Class<?> componentType, int modifiers, int length,
                               Object sourceArray, int sourceOffset) {
         // modifiers may be ignored for now
@@ -162,8 +163,7 @@ public static Object newInstance(Class<?> componentType, int modifiers, int leng
         Object newArray;
         if ((modifiers & NULL_CHECKED) != 0 &&
             length > 0 &&
-            componentType.isValue() &&
-            Modifier.isFinal(componentType.getModifiers())) {
+            ValueClass.isConcreteValueClass(componentType)) {
             newArray = ValueClass.newNullRestrictedAtomicArray(
                         componentType, length, get(sourceArray, 0));
         } else {

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

@@ -35,8 +35,10 @@
  * <p>Instances of classes that implement this interface must implement
  * an equals() method that equates any two null restricted type instances that
  * share the same base type.
+ *
+ * @since Valhalla
  */
-@PreviewFeature(feature = Feature.VALUE_OBJECTS, reflective = true)
+@PreviewFeature(feature = Feature.NULL_RESTRICTION, reflective = true)
 public interface NullRestrictedType extends Type {
     /**
      * {@return the {@code Type} object representing the type to which the null-restriction has

src/java.base/share/classes/java/lang/runtime/ArrayCreation.java

@@ -28,7 +28,6 @@
 import jdk.internal.javac.PreviewFeature;
 import jdk.internal.javac.PreviewFeature.Feature;
 import java.lang.reflect.Array;
-import java.lang.reflect.Modifier;
 import java.util.Arrays;
 import java.util.List;
 import java.util.Collections;
@@ -40,9 +39,11 @@
 
 /**
  * Factory and bootstrap methods for strictly-initialized array creation.
+ *
+ * @since Valhalla
  */
-@PreviewFeature(reflective = true, feature = Feature.VALUE_OBJECTS)
-public class ArrayCreation {
+@PreviewFeature(reflective = true, feature = Feature.NULL_RESTRICTION)
+public final class ArrayCreation {
 
     private ArrayCreation() {}
 

src/java.base/share/classes/jdk/internal/javac/PreviewFeature.java

@@ -67,6 +67,9 @@ public enum Feature {
         @JEP(number=401, title="Value Classes and Objects", status = "Preview")
         VALUE_OBJECTS,
 
+        @JEP(number=8303099, title="Null-Restricted and Nullable Types", status = "Preview")
+        NULL_RESTRICTION,
+
         // The JDK build process involves creating an interim javac which is then
         // used to compile the rest of the JDK. The jdk.internal.javac.PreviewFeature
         // annotation from the current sources is used when compiling interim javac.