Hi Shanliang,
On 30.9.2013 08:55, shanliang wrote:
Hi,
Please review this test fix, I set a much long waiting time to receive a
notification, but not simply wait the test timeout, in order to have
less modification.
IMO, the change to let the harness terminate the test on timeout was
quite readable as well. But you leave more than sufficient buffer for
any delays in delivering notifications so I suppose it should be fine.
You are missing the debugging echos in places where you extracted the
checkReceived() method.
And just a small nit - on line 31 you should probably add the @author tag.
Cheers,
-JB-
webrev: http://cr.openjdk.java.net/~sjiang/JDK-8025206/00/
bug: https://bugs.openjdk.java.net/browse/JDK-8025206
Thanks,
Shanliang
