Skip to content

Commit

Permalink
lib: replace var with let
Browse files Browse the repository at this point in the history
PR-URL: #30396
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
  • Loading branch information
Dennis Saenger authored and MylesBorins committed Dec 17, 2019
1 parent 2e37828 commit 1cfaccd
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/internal/v8_prof_polyfill.js
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ try {
const fd = fs.openSync(logFile, 'r');
const buf = Buffer.allocUnsafe(4096);
const dec = new (require('string_decoder').StringDecoder)('utf-8');
var line = '';
let line = '';

{
const message = versionCheck(peekline(), process.versions.v8);
Expand Down Expand Up @@ -125,7 +125,7 @@ function versionCheck(firstLine, expected) {
return 'Unable to read v8-version from log file.';
}
// Compare major, minor and build; ignore the patch and candidate fields.
for (var i = 0; i < 3; i++)
for (let i = 0; i < 3; i++)
if (curVer[i] !== firstLine[i + 1])
return 'Testing v8 version different from logging version';
}
Expand Down

0 comments on commit 1cfaccd

Please sign in to comment.