RFR: 8367561: Getting some "header" property from a file:// URL causes a file descriptor leak [v3]
Daniel Fuchs
dfuchs at openjdk.org
Mon Oct 13 11:18:07 UTC 2025
On Mon, 13 Oct 2025 11:15:16 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> Hello Volkan, I've now added a comment in the test methods to explain why this line is present.
>
> Technically the reachabilityFence should be in the finally clause of a try-finally to make sure it's not rearranged away by JIT - but I'm not sure how important this is for a test like this. Alternative could be to stash the reference in a static field. As long as the test methods are not run in parallel that should work too.
Otherwise LGTM
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27633#discussion_r2426004480
More information about the net-dev
mailing list