Thanks for the review Mike, -Joe
On 05/13/2013 10:06 PM, Mike Duigou wrote:
Looks fine to me. Welcome back Objects.requireNonNull(obj, Supplier)! Mike On May 10 2013, at 14: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 Thanks, -Joe
