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