>>>> (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-

Reply via email to