Code review request for JDK-8014365 Restore Objects.requireNonNull(T, Supplier<String>)
Joe Darcy
joe.darcy at oracle.com
Wed May 15 16:44:27 UTC 2013
On 05/14/2013 06:32 AM, Alan Bateman wrote:
> On 10/05/2013 22:01, Joe Darcy wrote:
>> Hello,
>>
>> Please (re)review this change to introduce Objects.requireNonNull(T,
>> Supplier<String>):
>>
>> http://cr.openjdk.java.net/~darcy/8014365.0/
>>
>> The original change had to be pulled out because of a build issue
>> (8012343: Objects.requireNonNull(Object,Supplier) breaks genstubs
>> build); I'll be asking for a review on build-dev of the build-related
>> change in langtools. The test portion of the patch is slightly
>> different than before because of the intervening changes made for
>>
>> 8013712: Add Objects.nonNull and Objects.isNull
> I realize this has already been pushed but just to point out a missing
> parenthesis on line 272 in the javadoc, needs to be ")}".
>
Sorry for introduce the javadoc issue.
Please review this patch
--- a/src/share/classes/java/util/Objects.java Mon May 13 22:16:55
2013 -0700
+++ b/src/share/classes/java/util/Objects.java Wed May 15 09:43:16
2013 -0700
@@ -269,7 +269,7 @@
* Checks that the specified object reference is not {@code null} and
* throws a customized {@link NullPointerException} if it is.
*
- * <p>Unlike the method {@link requireNonNull(Object, String},
+ * <p>Unlike the method {@link #requireNonNull(Object, String)},
* this method allows creation of the message to be deferred until
* after the null check is made. While this may confer a
* performance advantage in the non-null case, when deciding to
and I'll file a bug a push the fix.
Thanks,
-Joe
More information about the core-libs-dev
mailing list