Minimizing ops in res.send()

avoidwork · Jul 2, 2018 · 4399f51 · 4399f51
1 parent 04f69a6
commit 4399f51
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/lib/woodland.js b/lib/woodland.js
@@ -85,8 +85,8 @@ class Woodland extends Base {
 
 						partial(req, res, buffered, status, headers);
 						output = buffered.slice(req.range.start, req.range.end + 1).toString();
-					} else if (typeof output !== "string") {
-						output = "toString" in output ? output.toString() : output;
+					} else if (typeof output !== "string" && "toString" in output) {
+						output = output.toString();
 					}
 				}
 
@@ -109,7 +109,9 @@ class Woodland extends Base {
 					writeHead(res, status, headers);
 					output.on("error", () => void 0).pipe(res);
 				} else {
-					output = typeof output === "string" ? output : "toString" in output ? output.toString() : output;
+					if (typeof output !== "string" && "toString" in output) {
+						output = output.toString();
+					}
 
 					if (req.headers.range !== void 0) {
 						const buffered = Buffer.from(output);

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "woodland",
-  "version": "7.1.2",
+  "version": "7.1.3",
   "description": "Lightweight HTTP/HTTP2 router with automatic `Allow` & `CORS` headers",
   "main": "index.js",
   "scripts": {