On 3/30/2016 5:34 PM, Mandy Chung wrote:
On Mar 30, 2016, at 4:48 PM, Joseph D. Darcy <[email protected]> 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
Pushed after a de-tabbification and verifying the set of tests to run was the same before and after the update.
Thanks, -Joe
