On Fri, 20 Mar 2026 12:22:23 GMT, Jaikiran Pai <[email protected]> wrote:

> Hello Eirik, I've updated the JBS issue title to fix a typo as well as change 
> it to a bug. Please update the title of the PR to match the JBS issue.

Thanks Jai, I have also updated JDK-8380450 to be a bug, tweaked the title and 
included a reproducer.

> If any of that code throws something unexpected then it will get propagated 
> as an exception that results in a test failure, like any other test. So I 
> think the `assertDoesNotThrow()` is not needed here, isn't it?

My initial fix caused writeCEN to fail with an exception, during close as a 
suppressed exception. But you're right that we could let this propagate 
instead. I'll fix.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/30319#issuecomment-4097720421
PR Review Comment: https://git.openjdk.org/jdk/pull/30319#discussion_r2965558024

Reply via email to