On Mon, 21 Dec 2020 07:54:17 GMT, Andrey Turbanov <github.com+741251+turban...@openjdk.org> wrote:
> Cleanup code to use new handy methods in > `java.io.InputStream`/`java.nio.file.Files` instead of manual stream copy: > 1. java.io.InputStream#readAllBytes > 2. java.io.InputStream#transferTo > 3. java.nio.file.Files#copy > > Similar issue - https://bugs.openjdk.java.net/browse/JDK-8080272 src/java.desktop/windows/classes/sun/print/Win32PrintJob.java line 436: > 434: if (mDestination != null) { // if destination attribute is > set > 435: try { > 436: Files.copy(instream, Path.of(mDestination), > StandardCopyOption.REPLACE_EXISTING); Don't we need to close the instream in finally block? ------------- PR: https://git.openjdk.java.net/jdk/pull/1856
