From 230478592dea53568059566bbd86084f792a2e2e Mon Sep 17 00:00:00 2001 From: Mike Duigou Date: Fri, 27 Sep 2013 17:27:15 -0700 Subject: [PATCH] 8025610: Add explicit @throws NPE documentation to Optional constructor and Optional.of Reviewed-by: briangoetz, chegar, alanb --- jdk/src/share/classes/java/util/Optional.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/jdk/src/share/classes/java/util/Optional.java b/jdk/src/share/classes/java/util/Optional.java index ce2a33d1de6..10b79234b18 100644 --- a/jdk/src/share/classes/java/util/Optional.java +++ b/jdk/src/share/classes/java/util/Optional.java @@ -85,6 +85,7 @@ public final class Optional { * Constructs an instance with the value present. * * @param value the non-null value to be present + * @throws NullPointerException if value is null */ private Optional(T value) { this.value = Objects.requireNonNull(value); @@ -96,6 +97,7 @@ public final class Optional { * @param the class of the value * @param value the value to be present, which must be non-null * @return an {@code Optional} with the value present + * @throws NullPointerException if value is null */ public static Optional of(T value) { return new Optional<>(value);