debugger: refactor _debugger.js

* `==` -> `===`
* use white space in array to improve readability
PR-URL: #9860
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

Author: Trott

lib/_debugger.js

@@ -187,7 +187,7 @@ Client.prototype._addScript = function(desc) {
 this.scripts[desc.id]=desc;
 if(desc.name){
 desc.isNative=(desc.name.replace('.js','')innatives)||
-desc.name=='node.js';
+desc.name==='node.js';
 }
 };
 
@@ -202,7 +202,7 @@ Client.prototype._onResponse = function(res) {
 varindex=-1;
 
 this._reqCallbacks.some(function(fn,i){
-if(fn.request_seq==res.body.request_seq){
+if(fn.request_seq===res.body.request_seq){
 cb=fn;
 index=i;
 returntrue;
@@ -212,25 +212,25 @@ Client.prototype._onResponse = function(res) {
 varself=this;
 varhandled=false;
 
-if(res.headers.Type=='connect'){
+if(res.headers.Type==='connect'){
 // Request a list of scripts for our own storage.
 self.reqScripts();
 self.emit('ready');
 handled=true;
 
-}elseif(res.body&&res.body.event=='break'){
+}elseif(res.body&&res.body.event==='break'){
 this.emit('break',res.body);
 handled=true;
 
-}elseif(res.body&&res.body.event=='exception'){
+}elseif(res.body&&res.body.event==='exception'){
 this.emit('exception',res.body);
 handled=true;
 
-}elseif(res.body&&res.body.event=='afterCompile'){
+}elseif(res.body&&res.body.event==='afterCompile'){
 this._addHandle(res.body.body.script);
 handled=true;
 
-}elseif(res.body&&res.body.event=='scriptCollected'){
+}elseif(res.body&&res.body.event==='scriptCollected'){
 // ???
 this._removeScript(res.body.body.script);
 handled=true;
@@ -328,7 +328,7 @@ Client.prototype.reqScopes = function(cb) {
 Client.prototype.reqEval=function(expression,cb){
 varself=this;
 
-if(this.currentFrame==NO_FRAME){
+if(this.currentFrame===NO_FRAME){
 // Only need to eval in global scope.
 this.reqFrameEval(expression,NO_FRAME,cb);
 return;
@@ -358,7 +358,7 @@ Client.prototype.reqEval = function(expression, cb) {
 
 // Finds the first scope in the array in which the expression evals.
 Client.prototype._reqFramesEval=function(expression,evalFrames,cb){
-if(evalFrames.length==0){
+if(evalFrames.length===0){
 // Just eval in global scope.
 this.reqFrameEval(expression,NO_FRAME,cb);
 return;
@@ -382,7 +382,7 @@ Client.prototype.reqFrameEval = function(expression, frame, cb) {
 arguments: {expression: expression}
 };
 
-if(frame==NO_FRAME){
+if(frame===NO_FRAME){
 req.arguments.global=true;
 }else{
 req.arguments.frame=frame;
@@ -529,9 +529,9 @@ Client.prototype.mirrorObject = function(handle, depth, cb) {
 varmirror;
 varwaiting=1;
 
-if(handle.className=='Array'){
+if(handle.className==='Array'){
 mirror=[];
-}elseif(handle.className=='Date'){
+}elseif(handle.className==='Date'){
 mirror=newDate(handle.value);
 }else{
 mirror={};
@@ -774,10 +774,20 @@ function Interface(stdin, stdout, args) {
 process.once('SIGHUP',process.exit.bind(process,0));
 
 varproto=Interface.prototype;
-constignored=['pause','resume','exitRepl','handleBreak',
-'requireConnection','killChild','trySpawn',
-'controlEval','debugEval','print','childPrint',
-'clearline'];
+constignored=[
+'pause',
+'resume',
+'exitRepl',
+'handleBreak',
+'requireConnection',
+'killChild',
+'trySpawn',
+'controlEval',
+'debugEval',
+'print',
+'childPrint',
+'clearline'
+];
 constshortcut={
 'run': 'r',
 'cont': 'c',
@@ -1097,14 +1107,14 @@ Interface.prototype.list = function(delta) {
 varlineno=res.fromLine+i+1;
 if(lineno<from||lineno>to)continue;
 
-constcurrent=lineno==1+client.currentSourceLine;
+constcurrent=lineno===1+client.currentSourceLine;
 constbreakpoint=client.breakpoints.some(function(bp){
 return(bp.scriptReq===client.currentScript||
 bp.script===client.currentScript)&&
-bp.line==lineno;
+bp.line===lineno;
 });
 
-if(lineno==1){
+if(lineno===1){
 // The first line needs to have the module wrapper filtered out of
 // it.
 varwrapper=Module.wrapper[0];
@@ -1151,7 +1161,7 @@ Interface.prototype.backtrace = function() {
 return;
 }
 
-if(bt.totalFrames==0){
+if(bt.totalFrames===0){
 self.print('(empty stack)');
 }else{
 consttrace=[];
@@ -1193,10 +1203,10 @@ Interface.prototype.scripts = function() {
 varscript=client.scripts[id];
 if(script!==null&&typeofscript==='object'&&script.name){
 if(displayNatives||
-script.name==client.currentScript||
+script.name===client.currentScript||
 !script.isNative){
 scripts.push(
-(script.name==client.currentScript ? '* ' : ' ')+
+(script.name===client.currentScript ? '* ' : ' ')+
 id+': '+
 path.basename(script.name)
 );