Commit 583f334
debugger: remove variable redeclarations
Some variables are declared with var more than once in the same scope.
This change reduces the declarations to one per scope.
PR-URL: #4633
Reviewed-By: jasnell - James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>1 parent 7821b3e commit 583f334
1 file changed
Lines changed: 5 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -462,7 +462,7 @@ Client.prototype.setBreakpoint = function(req, cb) { | |||
| 462 | 462 | }; | |
| 463 | 463 | ||
| 464 | 464 | Client.prototype.clearBreakpoint = function(req, cb) { | |
| 465 | - var req = { | ||
| 465 | + req = { | ||
| 466 | 466 | command: 'clearbreakpoint', | |
| 467 | 467 | arguments: req | |
| 468 | 468 | }; | |
@@ -1345,9 +1345,10 @@ Interface.prototype.setBreakpoint = function(script, line, | |||
| 1345 | 1345 | return; | |
| 1346 | 1346 | } | |
| 1347 | 1347 | ||
| 1348 | + let req; | ||
| 1348 | 1349 | if (/\(\)$/.test(script)) { | |
| 1349 | 1350 | // setBreakpoint('functionname()'); | |
| 1350 | - var req = { | ||
| 1351 | + req = { | ||
| 1351 | 1352 | type: 'function', | |
| 1352 | 1353 | target: script.replace(/\(\)$/, ''), | |
| 1353 | 1354 | condition: condition | |
@@ -1373,7 +1374,6 @@ Interface.prototype.setBreakpoint = function(script, line, | |||
| 1373 | 1374 | if (ambiguous) return this.error('Script name is ambiguous'); | |
| 1374 | 1375 | if (line <= 0) return this.error('Line should be a positive value'); | |
| 1375 | 1376 | ||
| 1376 | - var req; | ||
| 1377 | 1377 | if (scriptId) { | |
| 1378 | 1378 | req = { | |
| 1379 | 1379 | type: 'scriptId', | |
@@ -1630,7 +1630,7 @@ Interface.prototype.trySpawn = function(cb) { | |||
| 1630 | 1630 | ||
| 1631 | 1631 | var isRemote = false; | |
| 1632 | 1632 | if (this.args.length === 2) { | |
| 1633 | - var match = this.args[1].match(/^([^:]+):(\d+)$/); | ||
| 1633 | + const match = this.args[1].match(/^([^:]+):(\d+)$/); | ||
| 1634 | 1634 | ||
| 1635 | 1635 | if (match) { | |
| 1636 | 1636 | // Connecting to remote debugger | |
@@ -1654,7 +1654,7 @@ Interface.prototype.trySpawn = function(cb) { | |||
| 1654 | 1654 | } | |
| 1655 | 1655 | isRemote = true; | |
| 1656 | 1656 | } else { | |
| 1657 | - var match = this.args[1].match(/^--port=(\d+)$/); | ||
| 1657 | + const match = this.args[1].match(/^--port=(\d+)$/); | ||
| 1658 | 1658 | if (match) { | |
| 1659 | 1659 | // Start debugger on custom port | |
| 1660 | 1660 | // `node debug --port=5858 app.js` | |
0 commit comments