module: move unnecessary work for early return

The exts and trailingSlash variables are only used if the path isn't cached. This commit moves them further down in the code, and changes from var to const. PR-URL: #3579 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Trevor Norris <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>

Author: zertosh

lib/module.js

@@ -126,19 +126,18 @@ function tryExtensions(p, exts){
 
 varwarned=false;
 Module._findPath=function(request,paths){
-varexts=Object.keys(Module._extensions);
-
 if(path.isAbsolute(request)){
 paths=[''];
 }
 
-vartrailingSlash=(request.slice(-1)==='/');
-
 varcacheKey=JSON.stringify({request: request,paths: paths});
 if(Module._pathCache[cacheKey]){
 returnModule._pathCache[cacheKey];
 }
 
+constexts=Object.keys(Module._extensions);
+consttrailingSlash=request.slice(-1)==='/';
+
 // For each path
 for(vari=0,PL=paths.length;i<PL;i++){
 // Don't search further if path doesn't exist