Hi,

Please review a fix on synchronization. The real change is the double-checked locking, all others are s/Token.categories/tmpMap/g.

All JAXP tests passed. Before the fix, the test attached to JBS failed only once for me after 10,000 runs and then a few 1000 runs (impossible to get one with only 20 runs as the comment mentioned). The author of the original report put the actual pass rate at 99.994%. After the fix then, in theory it should be 100%, I've made a 100,000 run without seeing a failure.

JBS: https://bugs.openjdk.java.net/browse/JDK-8198548
webrevs: http://cr.openjdk.java.net/~joehw/jdk11/8198548/webrev/

Thanks,
Joe

Reply via email to