> On Mar 30, 2016, at 4:48 PM, Joseph D. Darcy <joe.da...@oracle.com> wrote: > > Hi Mandy, > > Hopefully the third time will be the charm for this changeset after your > correction to the commented-out test: > > http://cr.openjdk.java.net/~darcy/8151763.2 > > I aligned the bug number in column 64 unless the test name took more > characters. (This isn't as evident in the webrev since the tab expansion is > different than in a text editor.) >
Thanks for doing it. Looks fine with me. Just to mention it: these few lines are somewhat strange (shorter test name has more whitespace) that you may want to double check. Ok to push what you have. ! java/nio/file/WatchService/Basic.java 7158947 solaris-all Solaris 11 ! java/nio/file/WatchService/MayFlies.java 7158947 solaris-all Solaris 11 ! java/nio/file/WatchService/LotsOfEvents.java 7158947 solaris-all Solaris 11 Mandy