On Wed, 28 Aug 2024 18:35:32 GMT, Brent Christian <bchri...@openjdk.org> wrote:
>> The InterruptibleIO feature (which was only on Solaris) was removed some >> time ago. See [JDK-4385444](https://bugs.openjdk.org/browse/JDK-4385444). >> >> Vestiges of it remain in jdk.internal.loader.Resource.getBytes(), and should >> be removed. > > Brent Christian has updated the pull request incrementally with one > additional commit since the last revision: > > move getContentLength() inside try block I think this looks okay. I suspect the catching of IOException from close can be removed too but we don't have to do that now. ------------- Marked as reviewed by alanb (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/20736#pullrequestreview-2267010682
