<AWT Dev> RFR: 8262161 Refactor manual I/O stream copying to new convinient methods in java.desktop

Prasanta Sadhukhan psadhukhan at openjdk.java.net
Tue Feb 23 15:49:44 UTC 2021


On Mon, 21 Dec 2020 07:54:17 GMT, Andrey Turbanov <github.com+741251+turbanoff at 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


More information about the awt-dev mailing list