[jdk8u-dev] RFR: 8274893: Update java.desktop classes to use try-with-resources

Andrey Turbanov aturbanov at openjdk.org
Mon Sep 1 11:53:48 UTC 2025


On Mon, 1 Sep 2025 11:00:23 GMT, Timofei Pushkin <tpushkin at openjdk.org> wrote:

>> jdk/src/share/classes/com/sun/media/sound/JARSoundbankReader.java line 91:
>> 
>>> 89:         if (stream == null)
>>> 90:             return null;
>>> 91:         try {
>> 
>> There should be try-with-resources
>
> There was in the original change but in this JDK try-with-resources on an existing variable is not possible (no JDK-7196163), so I fell back to try-finally. This is not the only place where I've done this, see [832c3c3](https://github.com/openjdk/jdk8u-dev/pull/686/commits/832c3c3bb4c132b025e7d3043b40be0d2d025a95).
> 
> Alternatives are to either create a copy variable for try-with-resources (which also must be used inside it to prevent a compiler warning) or move all interactions with the variable inside try-with-resources — both diverge code from the original change and the pre-existing code so I decided against them.

Shouldn't we revert formatting changes here, then?

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

PR Review Comment: https://git.openjdk.org/jdk8u-dev/pull/686#discussion_r2313743255


More information about the jdk8u-dev mailing list