Merge pull request #27043 from geoand/transfer-to

Use InputStream.transferTo instead of manually using a byte array
quarkusio · Aug 1, 2022 · 34f683a · 34f683a
2 parents 58f9efc + 711256e
commit 34f683a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/...c/main/java/org/jboss/resteasy/reactive/common/providers/serialisers/FileBodyHandler.java b/...c/main/java/org/jboss/resteasy/reactive/common/providers/serialisers/FileBodyHandler.java
@@ -36,11 +36,7 @@ public File readFrom(Class<File> type, Type genericType,
         }
 
         try (OutputStream output = new BufferedOutputStream(new FileOutputStream(downloadedFile))) {
-            int read;
-            final byte[] buf = new byte[2048];
-            while ((read = entityStream.read(buf)) != -1) {
-                output.write(buf, 0, read);
-            }
+            entityStream.transferTo(output);
         }
 
         return downloadedFile;

diff --git a/...g/jboss/resteasy/reactive/common/providers/serialisers/InputStreamMessageBodyHandler.java b/...g/jboss/resteasy/reactive/common/providers/serialisers/InputStreamMessageBodyHandler.java
@@ -34,11 +34,7 @@ public void writeTo(InputStream inputStream, Class<?> type, Type genericType, An
 
     protected void writeTo(InputStream inputStream, OutputStream entityStream) throws IOException {
         try {
-            byte[] buffer = new byte[8192];
-            int c;
-            while ((c = inputStream.read(buffer)) != -1) {
-                entityStream.write(buffer, 0, c);
-            }
+            inputStream.transferTo(entityStream);
         } finally {
             try {
                 inputStream.close();