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