Review request: 8004042 : Arrrghs.java test failed on windows with access error.

Stuart Marks stuart.marks at oracle.com
Fri Dec 7 16:22:16 UTC 2012


On 12/7/12 7:38 AM, David DeHaven wrote:
>> (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.

Did you need somebody to push this for you?

s'marks



More information about the core-libs-dev mailing list