Looks good. Hopefully Mandy can give the thumbs up before you push.

Approved.

regards,
Sean.

On 04/12/2014 07:29, Ivan Gerasimov wrote:


On 04.12.2014 2:44, Mandy Chung wrote:

On 12/3/2014 1:00 PM, Ivan Gerasimov wrote:
WEBREV: http://cr.openjdk.java.net/~igerasim/8065765/0/webrev/

Another alternative is to use Files.newDirectoryStream(path, "*.jar")
and increment the count.  What you have is okay too.

Yes, this is much shorter and cleaner.

Here's the updated webrev:
http://cr.openjdk.java.net/~igerasim/8065765/1/webrev/

Sincerely yours,
Ivan


Reply via email to