reduce allocations in transferTo()

robaho · robaho · commit 42f7413db878 · 2025-01-03T08:34:57.000-06:00
diff --git a/src/main/java/robaho/net/httpserver/NoSyncBufferedInputStream.java b/src/main/java/robaho/net/httpserver/NoSyncBufferedInputStream.java
@@ -262,6 +262,15 @@ public long transferTo(OutputStream out) throws IOException {
             out.write(buf,pos,count-pos);
             pos = count;
         }
-        return super.transferTo(out);
+        long total = avail;
+        while (true) {
+            fill();
+            if (count <= 0) {
+                break;
+            }
+            out.write(buf, 0, count);
+            total += count;
+        }
+        return total;
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -262,6 +262,15 @@ public long transferTo(OutputStream out) throws IOException {`
`262`	`262`	`out.write(buf,pos,count-pos);`
`263`	`263`	`pos = count;`
`264`	`264`	`}`
`265`		`- return super.transferTo(out);`
	`265`	`+ long total = avail;`
	`266`	`+ while (true) {`
	`267`	`+ fill();`
	`268`	`+ if (count <= 0) {`
	`269`	`+ break;`
	`270`	`+ }`
	`271`	`+ out.write(buf, 0, count);`
	`272`	`+ total += count;`
	`273`	`+ }`
	`274`	`+ return total;`
`266`	`275`	`}`
`267`	`276`	`}`