From be4744a8c0ac52a5417d835cc72d92397c377d55 Mon Sep 17 00:00:00 2001 From: Andres Suarez Date: Wed, 28 Oct 2015 20:09:38 -0700 Subject: [PATCH] module: move unnecessary work for early return `exts` and `trailingSlash` are only used if the path isn't cached. --- lib/module.js | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/lib/module.js b/lib/module.js index 82b1971e8bf009..e403c1f73bc190 100644 --- a/lib/module.js +++ b/lib/module.js @@ -136,19 +136,18 @@ function tryExtensions(p, exts) { var warned = false; Module._findPath = function(request, paths) { - var exts = Object.keys(Module._extensions); - if (path.isAbsolute(request)) { paths = ['']; } - var trailingSlash = (request.slice(-1) === '/'); - var cacheKey = JSON.stringify({request: request, paths: paths}); if (Module._pathCache[cacheKey]) { return Module._pathCache[cacheKey]; } + const exts = Object.keys(Module._extensions); + const trailingSlash = request.slice(-1) === '/'; + // For each path for (var i = 0, PL = paths.length; i < PL; i++) { // Don't search further if path doesn't exist