On 6/21/18 11:59 AM, Roger Riggs wrote:
Please review a test improvement to avoid spurious errors caused by concurrent changes in open files during the test. The other existing conditions in the test are sufficient to verify the correct finalize behavior. Some additional diagnostics are added and the tests removed from the problem list.

Webrev:
http://cr.openjdk.java.net/~rriggs/webrev-count-wrong-8202292/index.html

The fix looks reasonable.  It's good to get this test in service again.

Do you want to keep the commented lines in the test? Maybe add a comment instead.

Mandy

Reply via email to