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