Will fix, not used to see @test before imports. Same problem with 6849945.
Erik
Yekaterina Kantserova skrev 2013-10-30 15:40:
Not a reviewer, so just en observation. Seems like there are 2 jtreg
headers now:
24 /*
25 * @test
26 * @bug 6222826
27 * @summary Test that each thread in the thread pool runs
28 * in the context of the monitor.start() caller.
29 * @author Luis-Miguel Alventosa
30 * @run clean ThreadPoolAccTest
31 * @run build ThreadPoolAccTest
32 * @run main ThreadPoolAccTest
33 */
and
49 /*
50 * @test
51 *
52 * @run main/othervm ThreadPoolAccTest
53 */
The original one does not use "other" in @run.
Thanks,
Katja
On 10/30/2013 03:13 PM, Erik Gahlin wrote:
Could I have a review of this intermittently failing test.
Instead of doing Thread.sleep(2000) and assume principal is
visible/set the test now spins until principal is available. I also
cleaned up code to avoid warnings.
Thanks
Erik
Webrev:
http://cr.openjdk.java.net/~egahlin/8027209_2/
Bug:
https://bugs.openjdk.java.net/browse/JDK-8027209