Code review request for JDK-8014365 Restore Objects.requireNonNull(T, Supplier) (original) (raw)
Alan Bateman Alan.Bateman at oracle.com
Tue May 14 13:32:52 UTC 2013
- Previous message: Code review request for JDK-8014365 Restore Objects.requireNonNull(T, Supplier)
- Next message: Code review request for JDK-8014365 Restore Objects.requireNonNull(T, Supplier)
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 10/05/2013 22:01, Joe Darcy wrote:
Hello,
Please (re)review this change to introduce Objects.requireNonNull(T, Supplier): 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.
- Previous message: Code review request for JDK-8014365 Restore Objects.requireNonNull(T, Supplier)
- Next message: Code review request for JDK-8014365 Restore Objects.requireNonNull(T, Supplier)
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]