Review request: 8004042 : Arrrghs.java test failed on windows with access error. (original) (raw)

David DeHaven david.dehaven at oracle.com
Fri Dec 7 18:09:04 UTC 2012

(There's another issue which is that if there were previous retries, the ADEs from them are thrown away. But maybe we should save that one for another day.)

I had the same thought, but aside from collecting and reporting all of them somehow I'm not sure what could be done about it. Maybe instead of: cause = ade; do: if (cause != null) { cause.addSuppressed(ade); } else { cause = ade; } Then they'll at least all be reported when RuntimeException is thrown. Ah, ok, this isn't bad at all! Let's go with this. Sounds like a plan :) Did you need somebody to push this for you? Kumar is going to push for me.

I updated the webrev, unfortunately I lost count and updated the same webrev so it's still at webrev.2. I'm doing a final sanity check and will submit a JPRT run when that's done.

http://cr.openjdk.java.net/~ddehaven/8004042/webrev.2/

-DrD-

More information about the core-libs-dev mailing list