module: refactor redeclared variable

Trott · rvagg · commit c09eb44a5918 · 2016-02-08T23:01:05.000+11:00
`homedir` was declared with `var` twice in the same scope in `lib/module.js`. This change makes it a single declaration. PR-URL: #4962 Reviewed-By: Brian White <mscdex@mscdex.net> Reviewed-By: Minwoo Jung <jmwsoft@gmail.com> Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
diff --git a/lib/module.js b/lib/module.js
@@ -452,10 +452,11 @@ Module.runMain = function(){
 Module._initPaths = function(){
 const isWindows = process.platform === 'win32';
 
+ var homeDir;
 if (isWindows){
- var homeDir = process.env.USERPROFILE;
+ homeDir = process.env.USERPROFILE;
 } else{
- var homeDir = process.env.HOME;
+ homeDir = process.env.HOME;
 }
 
 var paths = [path.resolve(process.execPath, '..', '..', 'lib', 'node')];

-Original file line number
+Diff line change
 Module._initPaths=function(){
 constisWindows=process.platform==='win32';
 +varhomeDir;
 if(isWindows){
 -varhomeDir=process.env.USERPROFILE;
 +homeDir=process.env.USERPROFILE;
 }else{
 -varhomeDir=process.env.HOME;
 +homeDir=process.env.HOME;
+}
 varpaths=[path.resolve(process.execPath,'..','..','lib','node')];