Thanks. Any further comments from anyone else?
Brian On Oct 10, 2013, at 9:04 PM, David Holmes wrote: > Ship it! :) > > 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 >>
