Re: RFR: JDK-8012650 and JDK-8011918

2013-04-25 Thread Alan Bateman
On 25/04/2013 19:14, Henry Jen wrote: : Both fixed, new webrev is at http://cr.openjdk.java.net/~henryjen/tl/8012650-8011918.1 Cheers, Henry Thanks, I didn't mean you have to have to generate a new webrev for such minor matters. Anyway, I don't see any other issues and looks fine to me. -Al

Re: RFR: JDK-8012650 and JDK-8011918

2013-04-25 Thread Henry Jen
On 04/24/2013 06:54 AM, Alan Bateman wrote: > On 24/04/2013 00:09, Henry Jen wrote: >> Hi, >> >> Please review static Stream factory methods and methods for Arrays. >> Webrev is at >> >> http://cr.openjdk.java.net/~henryjen/tl/8012650-8011918.0 >> >> Stream methods are covered in a test suite which

Re: RFR: JDK-8012650 and JDK-8011918

2013-04-24 Thread Alan Bateman
On 24/04/2013 00:09, Henry Jen wrote: Hi, Please review static Stream factory methods and methods for Arrays. Webrev is at http://cr.openjdk.java.net/~henryjen/tl/8012650-8011918.0 Stream methods are covered in a test suite which will be put back soon once all depends part is in place. Arrays

RFR: JDK-8012650 and JDK-8011918

2013-04-23 Thread Henry Jen
Hi, Please review static Stream factory methods and methods for Arrays. Webrev is at http://cr.openjdk.java.net/~henryjen/tl/8012650-8011918.0 Stream methods are covered in a test suite which will be put back soon once all depends part is in place. Arrays setAll and parallelSetAll methods has t