Code review request for JDK-8014365 Restore Objects.requireNonNull(T, Supplier<String>)

Alan Bateman Alan.Bateman at oracle.com
Tue May 14 13:32:52 UTC 2013


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 ")}".

-Alan.



More information about the core-libs-dev mailing list