feat(perf): hoist one more function

TheLarkInn · TheLarkInn · commit 082a60579128 · 2017-06-30T15:07:02.000-05:00
diff --git a/lib/RequestShortener.js b/lib/RequestShortener.js
@@ -18,6 +18,9 @@ const shortenPath = (path) => {
 	return path.replace(PATH_CHARS_REGEXP, "\\$&");
 };
 
+const createRegExpForTypeWith = (regexpTypePartial) => {
+	return new RegExp(`^${regexpTypePartial}|(!)${regexpTypePartial}`, "g");
+};
 
 class RequestShortener {
 	constructor(directory) {
@@ -26,23 +29,23 @@ class RequestShortener {
 
 		if(directory) {
 			const currentDirectoryRegExpString = shortenPath(directory);
-			this.currentDirectoryRegExp = new RegExp("^" + currentDirectoryRegExpString + "|(!)" + currentDirectoryRegExpString, "g");
+			this.currentDirectoryRegExp = createRegExpForTypeWith(currentDirectoryRegExpString);
 		}
 
 		const dirname = path.dirname(directory);
 		const endsWithSeperator = SEPERATOR_REGEXP.test(dirname);
 		const parentDirectory = endsWithSeperator ? dirname.substr(0, dirname.length - 1) : dirname;
 		if(parentDirectory && parentDirectory !== directory) {
 			const parentDirectoryRegExpString = shortenPath(parentDirectory);
-			this.parentDirectoryRegExp = new RegExp("^" + parentDirectoryRegExpString + "|(!)" + parentDirectoryRegExpString, "g");
+			this.parentDirectoryRegExp = createRegExpForTypeWith(parentDirectoryRegExpString);
 		}
 
 		if(__dirname.length >= 2) {
 			const buildins = normalizeBackSlashDirection(path.join(__dirname, ".."));
 			const buildinsAsModule = this.currentDirectoryRegExp && this.currentDirectoryRegExp.test(buildins);
 			this.buildinsAsModule = buildinsAsModule;
 			const buildinsRegExpString = shortenPath(buildins);
-			this.buildinsRegExp = new RegExp("^" + buildinsRegExpString + "|(!)" + buildinsRegExpString, "g");
+			this.buildinsRegExp = createRegExpForTypeWith(buildinsRegExpString);
 		}
 
 		this.indexJsRegExp = /\/index.js(!|\?|\(query\))/g;
