Thanks!

Paul

On 1/10/11 10:59 AM, Mandy Chung wrote:
 On 1/9/11 5:10 PM, Paul Hohensee wrote:
Thanks to all who reviewed the previous webrev. Revised versions are here

http://cr.openjdk.java.net/~phh/6173675/webrev.02/

Looks good.  Thanks for updating it and adding the tests.

As for the tests, the existing java/lang/management regression tests definitely need upgrade and improvement. They were written before java.util.concurrent locks were added in 1.5 and also the tests have run into some timing issue between the work done by threads and the test verification. I'm fine for pushing the tests you have and then follow up with a CR as David suggests.

Thanks
Mandy

Reply via email to