Replacing home brewed arity detection for Function.prototype.length

avoidwork · Oct 20, 2017 · e832c1d · e832c1d
1 parent fc3bf50
commit e832c1d
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 5 deletions.
diff --git a/lib/utility.js b/lib/utility.js
@@ -2,6 +2,5 @@
 
 module.exports = {
 	escape: arg => arg.replace(/[-[\]\/{}()*+?.,\\^$|#]/g, "\\$&"),
-	getArity: arg => arg.toString().replace(/(^.*\()|(\).*)|(\n.*)/g, "").split(",").length,
 	schemeless: arg => arg.replace(/^.*:\/\//, "")
 };
diff --git a/lib/woodland.js b/lib/woodland.js
@@ -159,12 +159,12 @@ class Woodland {
 				process.nextTick(() => {
 					if (res.headersSent === false) {
 						let iter = middleware.next(),
-							arity = iter.done === false ? utility.getArity(iter.value) : 0;
+							arity = iter.done === false ? iter.value.length : 0;
 
 						if (iter.done === false) {
 							if (err !== void 0) {
 								do {
-									arity = utility.getArity(iter.value);
+									arity = iter.value.length;
 								} while (arity < 4 && (iter = middleware.next()) && iter.done === false);
 							}
 

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": "3.0.2",
+  "version": "3.0.3",
   "description": "Lightweight HTTP/HTTPS router with automatic `Allow` & `CORS` headers",
   "main": "index.js",
   "scripts": {