repl: upper case comments first char

Backport-PR-URL: #19244 PR-URL: #17919 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Weijia Wang <[email protected]> Reviewed-By: Khaidi Chu <[email protected]> Reviewed-By: Jon Moss <[email protected]> Reviewed-By: Lance Ball <[email protected]>

Author: BridgeAR

lib/repl.js

@@ -79,7 +79,7 @@ const kBufferedCommandSymbol = Symbol('bufferedCommand');
 constkContextId=Symbol('contextId');
 
 try{
-// hack for require.resolve("./relative") to work properly.
+// Hack for require.resolve("./relative") to work properly.
 module.filename=path.resolve('repl');
 }catch(e){
 // path.resolve('repl') fails when the current working directory has been
@@ -89,7 +89,7 @@ try{
 module.filename=path.resolve(dirname,'repl');
 }
 
-// hack for repl require to work properly with node_modules folders
+// Hack for repl require to work properly with node_modules folders
 module.paths=Module._nodeModulePaths(module.filename);
 
 // If obj.hasOwnProperty has been overridden, then calling
@@ -99,14 +99,12 @@ function hasOwnProperty(obj, prop){
 returnObject.prototype.hasOwnProperty.call(obj,prop);
 }
 
-
 // Can overridden with custom print functions, such as `probe` or `eyes.js`.
 // This is the default "writer" value if none is passed in the REPL options.
 exports.writer=util.inspect;
 
 exports._builtinLibs=internalModule.builtinLibs;
 
-
 functionREPLServer(prompt,
 stream,
 eval_,
@@ -149,7 +147,6 @@ function REPLServer(prompt,
 varself=this;
 
 self._domain=dom||domain.create();
-
 self.useGlobal=!!useGlobal;
 self.ignoreUndefined=!!ignoreUndefined;
 self.replMode=replMode||exports.REPL_MODE_SLOPPY;
@@ -162,7 +159,7 @@ function REPLServer(prompt,
 // Context id for use with the inspector protocol.
 self[kContextId]=undefined;
 
-// just for backwards compat, see github.com/joyent/node/pull/7127
+// Just for backwards compat, see github.com/joyent/node/pull/7127
 self.rli=this;
 
 constsavedRegExMatches=['','','','','','','','','',''];
@@ -187,8 +184,7 @@ function REPLServer(prompt,
 wrappedCmd=true;
 }
 
-// first, create the Script object to check the syntax
-
+// First, create the Script object to check the syntax
 if(code==='\n')
 returncb(null);
 
@@ -207,13 +203,13 @@ function REPLServer(prompt,
 }catch(e){
 debug('parse error %j',code,e);
 if(wrappedCmd){
+// Unwrap and try again
 wrappedCmd=false;
-// unwrap and try again
 code=input;
 wrappedErr=e;
 continue;
 }
-// preserve original error for wrapped command
+// Preserve original error for wrapped command
 consterror=wrappedErr||e;
 if(isRecoverableError(error,code))
 err=newRecoverable(error);
@@ -321,7 +317,7 @@ function REPLServer(prompt,
 if(!input&&!output){
 // legacy API, passing a 'stream'/'socket' option
 if(!stream){
-// use stdin and stdout as the default streams if none were given
+// Use stdin and stdout as the default streams if none were given
 stream=process;
 }
 if(stream.stdin&&stream.stdout){
@@ -371,7 +367,7 @@ function REPLServer(prompt,
 this.commands=Object.create(null);
 defineDefaultCommands(this);
 
-// figure out which "writer" function to use
+// Figure out which "writer" function to use
 self.writer=options.writer||exports.writer;
 
 if(options.useColors===undefined){
@@ -387,14 +383,14 @@ function REPLServer(prompt,
 }
 
 functionfilterInternalStackFrames(error,structuredStack){
-// search from the bottom of the call stack to
+// Search from the bottom of the call stack to
 // find the first frame with a null function name
 if(typeofstructuredStack!=='object')
 returnstructuredStack;
 constidx=structuredStack.reverse().findIndex(
 (frame)=>frame.getFunctionName()===null);
 
-// if found, get rid of it and everything below it
+// If found, get rid of it and everything below it
 structuredStack=structuredStack.splice(idx+1);
 returnstructuredStack;
 }
@@ -571,7 +567,7 @@ function REPLServer(prompt,
 return;
 }
 
-// editor mode
+// Editor mode
 if(key.ctrl&&!key.shift){
 switch(key.name){
 case'd': // End editor mode
@@ -591,7 +587,7 @@ function REPLServer(prompt,
 case'down': // Override next history item
 break;
 case'tab':
-// prevent double tab behavior
+// Prevent double tab behavior
 self._previousKey=null;
 ttyWrite(d,key);
 break;
@@ -861,10 +857,8 @@ function complete(line, callback){
 }
 
 varcompletions;
-
-// list of completion lists, one for each inheritance "level"
+// List of completion lists, one for each inheritance "level"
 varcompletionGroups=[];
-
 varcompleteOn,i,group,c;
 
 // REPL commands (e.g. ".break").
@@ -1042,7 +1036,7 @@ function complete(line, callback){
 }
 }
 }catch(e){
-//console.log("completion error walking prototype chain:" + e);
+//console.log("completion error walking prototype chain:" + e);
 }
 }
 
@@ -1085,7 +1079,7 @@ function complete(line, callback){
 }
 
 if(completionGroups.length){
-varuniq={};// unique completions across all groups
+varuniq={};// Unique completions across all groups
 completions=[];
 // Completion group 0 is the "closest"
 // (least far up the inheritance chain)
@@ -1100,7 +1094,7 @@ function complete(line, callback){
 uniq[c]=true;
 }
 }
-completions.push('');// separator btwn groups
+completions.push('');// Separator btwn groups
 }
 while(completions.length&&completions[completions.length-1]===''){
 completions.pop();
@@ -1167,7 +1161,7 @@ function _memory(cmd){
 self.lines=self.lines||[];
 self.lines.level=self.lines.level||[];
 
-// save the line so I can do magic later
+// Save the line so I can do magic later
 if(cmd){
 // TODO should I tab the level?
 constlen=self.lines.level.length ? self.lines.level.length-1 : 0;
@@ -1181,7 +1175,7 @@ function _memory(cmd){
 // Because I can not tell the difference between a } that
 // closes an object literal and a } that closes a function
 if(cmd){
-// going down is{and ( e.g. function(){
+// Going down is{and ( e.g. function(){
 // going up is } and )
 vardw=cmd.match(/{|\(/g);
 varup=cmd.match(/}|\)/g);
@@ -1192,8 +1186,8 @@ function _memory(cmd){
 if(depth){
 (functionworkIt(){
 if(depth>0){
-// going... down.
-// push the line#, depth count, and if the line is a function.
+// Going... down.
+// Push the line#, depth count, and if the line is a function.
 // Since JS only has functional scope I only need to remove
 // "function(){" lines, clearly this will not work for
 // "function()
@@ -1205,16 +1199,16 @@ function _memory(cmd){
 isFunction: /\bfunction\b/.test(cmd)
 });
 }elseif(depth<0){
-// going... up.
+// Going... up.
 varcurr=self.lines.level.pop();
 if(curr){
 vartmp=curr.depth+depth;
 if(tmp<0){
-//more to go, recurse
+// More to go, recurse
 depth+=curr.depth;
 workIt();
 }elseif(tmp>0){
-//remove and push back
+// Remove and push back
 curr.depth+=depth;
 self.lines.level.push(curr);
 }