On 11/10/2013 05:04, David Holmes wrote:
Ship it! :)
+1

-Chris.


Thanks,
David

On 11/10/2013 5:24 AM, Brian Burkhalter wrote:

On Oct 10, 2013, at 11:21 AM, Brian Burkhalter wrote:

On Oct 10, 2013, at 11:05 AM, Brian Burkhalter wrote:

Nit: In the test there are a few places where you have t on a line
by itself:

65 t);

but it can go on the previous line and not exceed the length of
other lines nearby. Also "+ should be " +

I don't see what you are referring to.

Oh, sorry, now I do. Will update.

I hope that as of now all nits have been picked.

http://cr.openjdk.java.net/~bpb/7179567/webrev.5/

Thanks,

Brian

Reply via email to