On Wed, 12 Feb 2025 16:47:29 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
>> Can I please get a review of this change in >> `jdk.tools.jlink.internal.plugins.ZipPlugin` which proposes to close the >> `Deflater` instance cleanly? This addresses >> https://bugs.openjdk.org/browse/JDK-8349907. >> >> As noted in that issue, the `Deflater` instance wasn't being closed in the >> exception code path of this method. The commit in this PR merely changes the >> code to use a try-with-resources block on the `Deflater` instance. >> >> No new tests have been introduced given the nature of the change. I've added >> a `noreg-hard` to the issue. Existing tier1, tier2 and tier3 tests continue >> to pass with this change without any related failures. > > Jaikiran Pai has updated the pull request incrementally with one additional > commit since the last revision: > > Chen's suggestion - include an assert in catch block to make clear that the > IOException isn't expected LGTM ------------- Marked as reviewed by sundar (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/23588#pullrequestreview-2614431315
