On 3/30/2016 5:34 PM, Mandy Chung wrote:
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
Pushed after a de-tabbification and verifying the set of tests to run
was the same before and after the update.
Thanks,
-Joe