Skip to content

Commit e37f510

Browse files
RamirezAlexcodebytere
authored andcommitted
benchmark: swap var for let in benchmarks
In benchmark directory this changes for loops using var to let when it applies for consistency PR-URL: #28958 Reviewed-By: Anna Henningsen <[email protected]>
1 parent c4613c6 commit e37f510

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

47 files changed

+51
-51
lines changed

benchmark/module/module-loader-deep.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ function main({ ext, cache, files }) {
1919
`${benchmarkDirectory}/a.js`,
2020
'module.exports = {};'
2121
);
22-
for (var i = 0; i <= files; i++) {
22+
for (let i = 0; i <= files; i++) {
2323
fs.mkdirSync(`${benchmarkDirectory}/${i}`);
2424
fs.writeFileSync(
2525
`${benchmarkDirectory}/${i}/package.json`,

benchmark/module/module-loader.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ const bench = common.createBenchmark(main, {
2121
function main({ n, name, cache, files, dir }) {
2222
tmpdir.refresh();
2323
fs.mkdirSync(benchmarkDirectory);
24-
for (var i = 0; i <= files; i++) {
24+
for (let i = 0; i <= files; i++) {
2525
fs.mkdirSync(`${benchmarkDirectory}${i}`);
2626
fs.writeFileSync(
2727
`${benchmarkDirectory}${i}/package.json`,

benchmark/os/cpus.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ const bench = common.createBenchmark(main, {
99

1010
function main({ n }) {
1111
bench.start();
12-
for (var i = 0; i < n; ++i)
12+
for (let i = 0; i < n; ++i)
1313
cpus();
1414
bench.end(n);
1515
}

benchmark/os/loadavg.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ const bench = common.createBenchmark(main, {
99

1010
function main({ n }) {
1111
bench.start();
12-
for (var i = 0; i < n; ++i)
12+
for (let i = 0; i < n; ++i)
1313
loadavg();
1414
bench.end(n);
1515
}

benchmark/os/networkInterfaces.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ const bench = common.createBenchmark(main, {
99

1010
function main({ n }) {
1111
bench.start();
12-
for (var i = 0; i < n; ++i)
12+
for (let i = 0; i < n; ++i)
1313
networkInterfaces();
1414
bench.end(n);
1515
}

benchmark/path/basename-posix.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ function main({ n, pathext }) {
2727
}
2828

2929
bench.start();
30-
for (var i = 0; i < n; i++) {
30+
for (let i = 0; i < n; i++) {
3131
posix.basename(i % 3 === 0 ? `${pathext}${i}` : pathext, ext);
3232
}
3333
bench.end(n);

benchmark/path/basename-win32.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ function main({ n, pathext }) {
2727
}
2828

2929
bench.start();
30-
for (var i = 0; i < n; i++) {
30+
for (let i = 0; i < n; i++) {
3131
win32.basename(i % 3 === 0 ? `${pathext}${i}` : pathext, ext);
3232
}
3333
bench.end(n);

benchmark/path/dirname-posix.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ const bench = common.createBenchmark(main, {
1717

1818
function main({ n, path }) {
1919
bench.start();
20-
for (var i = 0; i < n; i++) {
20+
for (let i = 0; i < n; i++) {
2121
posix.dirname(i % 3 === 0 ? `${path}${i}` : path);
2222
}
2323
bench.end(n);

benchmark/path/dirname-win32.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ const bench = common.createBenchmark(main, {
1717

1818
function main({ n, path }) {
1919
bench.start();
20-
for (var i = 0; i < n; i++) {
20+
for (let i = 0; i < n; i++) {
2121
win32.dirname(i % 3 === 0 ? `${path}${i}` : path);
2222
}
2323
bench.end(n);

benchmark/path/extname-posix.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ const bench = common.createBenchmark(main, {
2020

2121
function main({ n, path }) {
2222
bench.start();
23-
for (var i = 0; i < n; i++) {
23+
for (let i = 0; i < n; i++) {
2424
posix.extname(i % 3 === 0 ? `${path}${i}` : path);
2525
}
2626
bench.end(n);

benchmark/path/extname-win32.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ const bench = common.createBenchmark(main, {
2020

2121
function main({ n, path }) {
2222
bench.start();
23-
for (var i = 0; i < n; i++) {
23+
for (let i = 0; i < n; i++) {
2424
win32.extname(i % 3 === 0 ? `${path}${i}` : path);
2525
}
2626
bench.end(n);

benchmark/path/format-posix.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ function main({ n, props }) {
2020
};
2121

2222
bench.start();
23-
for (var i = 0; i < n; i++) {
23+
for (let i = 0; i < n; i++) {
2424
obj.base = `a${i}${props[2] || ''}`;
2525
obj.name = `a${i}${props[4] || ''}`;
2626
posix.format(obj);

benchmark/path/format-win32.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ function main({ n, props }) {
2020
};
2121

2222
bench.start();
23-
for (var i = 0; i < n; i++) {
23+
for (let i = 0; i < n; i++) {
2424
obj.base = `a${i}${props[2] || ''}`;
2525
obj.name = `a${i}${props[4] || ''}`;
2626
win32.format(obj);

benchmark/path/isAbsolute-posix.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ const bench = common.createBenchmark(main, {
1515

1616
function main({ n, path }) {
1717
bench.start();
18-
for (var i = 0; i < n; i++) {
18+
for (let i = 0; i < n; i++) {
1919
posix.isAbsolute(i % 3 === 0 ? `${path}${i}` : path);
2020
}
2121
bench.end(n);

benchmark/path/isAbsolute-win32.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ const bench = common.createBenchmark(main, {
1616

1717
function main({ n, path }) {
1818
bench.start();
19-
for (var i = 0; i < n; i++) {
19+
for (let i = 0; i < n; i++) {
2020
win32.isAbsolute(i % 3 === 0 ? `${path}${i}` : path);
2121
}
2222
bench.end(n);

benchmark/path/join-posix.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ function main({ n, paths }) {
1515
const orig = copy[1];
1616

1717
bench.start();
18-
for (var i = 0; i < n; i++) {
18+
for (let i = 0; i < n; i++) {
1919
if (i % 3 === 0) {
2020
copy[1] = `${orig}${i}`;
2121
posix.join(...copy);

benchmark/path/join-win32.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ function main({ n, paths }) {
1515
const orig = copy[1];
1616

1717
bench.start();
18-
for (var i = 0; i < n; i++) {
18+
for (let i = 0; i < n; i++) {
1919
if (i % 3 === 0) {
2020
copy[1] = `${orig}${i}`;
2121
win32.join(...copy);

benchmark/path/makeLong-win32.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ const bench = common.createBenchmark(main, {
1414

1515
function main({ n, path }) {
1616
bench.start();
17-
for (var i = 0; i < n; i++) {
17+
for (let i = 0; i < n; i++) {
1818
win32._makeLong(i % 3 === 0 ? `${path}${i}` : path);
1919
}
2020
bench.end(n);

benchmark/path/normalize-posix.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ const bench = common.createBenchmark(main, {
1616

1717
function main({ n, path }) {
1818
bench.start();
19-
for (var i = 0; i < n; i++) {
19+
for (let i = 0; i < n; i++) {
2020
posix.normalize(i % 3 === 0 ? `${path}${i}` : path);
2121
}
2222
bench.end(n);

benchmark/path/normalize-win32.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ const bench = common.createBenchmark(main, {
1616

1717
function main({ n, path }) {
1818
bench.start();
19-
for (var i = 0; i < n; i++) {
19+
for (let i = 0; i < n; i++) {
2020
win32.normalize(i % 3 === 0 ? `${path}${i}` : path);
2121
}
2222
bench.end(n);

benchmark/path/resolve-posix.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ function main({ n, paths }) {
1818
const orig = copy[0];
1919

2020
bench.start();
21-
for (var i = 0; i < n; i++) {
21+
for (let i = 0; i < n; i++) {
2222
if (i % 3 === 0) {
2323
copy[0] = `${orig}${i}`;
2424
posix.resolve(...copy);

benchmark/path/resolve-win32.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ function main({ n, paths }) {
1818
const orig = copy[0];
1919

2020
bench.start();
21-
for (var i = 0; i < n; i++) {
21+
for (let i = 0; i < n; i++) {
2222
if (i % 3 === 0) {
2323
copy[0] = `${orig}${i}`;
2424
win32.resolve(...copy);

benchmark/process/memoryUsage.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ const bench = common.createBenchmark(main, {
77

88
function main({ n }) {
99
bench.start();
10-
for (var i = 0; i < n; i++) {
10+
for (let i = 0; i < n; i++) {
1111
process.memoryUsage();
1212
}
1313
bench.end(n);

benchmark/process/next-tick-breadth-args.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ function main({ n }) {
3333
}
3434

3535
bench.start();
36-
for (var i = 0; i < n; i++) {
36+
for (let i = 0; i < n; i++) {
3737
if (i % 4 === 0)
3838
process.nextTick(cb4, 3.14, 1024, true, false);
3939
else if (i % 3 === 0)

benchmark/process/next-tick-breadth.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ function main({ n }) {
1515
}
1616

1717
bench.start();
18-
for (var i = 0; i < n; i++) {
18+
for (let i = 0; i < n; i++) {
1919
process.nextTick(cb);
2020
}
2121
}

benchmark/process/next-tick-exec-args.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ function main({ n }) {
1010
bench.end(n);
1111
}
1212

13-
for (var i = 0; i < n; i++) {
13+
for (let i = 0; i < n; i++) {
1414
if (i % 4 === 0)
1515
process.nextTick(onNextTick, i, true, 10, 'test');
1616
else if (i % 3 === 0)

benchmark/process/next-tick-exec.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ function main({ n }) {
1010
bench.end(n);
1111
}
1212

13-
for (var i = 0; i < n; i++) {
13+
for (let i = 0; i < n; i++) {
1414
process.nextTick(onNextTick, i);
1515
}
1616

benchmark/process/queue-microtask-breadth.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ function main({ n }) {
1515
}
1616

1717
bench.start();
18-
for (var i = 0; i < n; i++) {
18+
for (let i = 0; i < n; i++) {
1919
queueMicrotask(cb);
2020
}
2121
}

benchmark/querystring/querystring-stringify.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ function main({ type, n }) {
3838
querystring.stringify(inputs[name]);
3939

4040
bench.start();
41-
for (var i = 0; i < n; i += 1)
41+
for (let i = 0; i < n; i += 1)
4242
querystring.stringify(input);
4343
bench.end(n);
4444
}

benchmark/querystring/querystring-unescapebuffer.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ const bench = common.createBenchmark(main, {
1414

1515
function main({ input, n }) {
1616
bench.start();
17-
for (var i = 0; i < n; i += 1)
17+
for (let i = 0; i < n; i += 1)
1818
querystring.unescapeBuffer(input);
1919
bench.end(n);
2020
}

benchmark/string_decoder/string-decoder-create.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ const bench = common.createBenchmark(main, {
1111

1212
function main({ encoding, n }) {
1313
bench.start();
14-
for (var i = 0; i < n; ++i) {
14+
for (let i = 0; i < n; ++i) {
1515
const sd = new StringDecoder(encoding);
1616
!!sd.encoding;
1717
}

benchmark/timers/immediate.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ function breadth(N) {
6666
if (n === N)
6767
bench.end(N);
6868
}
69-
for (var i = 0; i < N; i++) {
69+
for (let i = 0; i < N; i++) {
7070
setImmediate(cb);
7171
}
7272
}
@@ -80,7 +80,7 @@ function breadth1(N) {
8080
if (n === N)
8181
bench.end(n);
8282
}
83-
for (var i = 0; i < N; i++) {
83+
for (let i = 0; i < N; i++) {
8484
setImmediate(cb, 1);
8585
}
8686
}
@@ -95,7 +95,7 @@ function breadth4(N) {
9595
if (n === N)
9696
bench.end(n);
9797
}
98-
for (var i = 0; i < N; i++) {
98+
for (let i = 0; i < N; i++) {
9999
setImmediate(cb, 1, 2, 3, 4);
100100
}
101101
}
@@ -107,7 +107,7 @@ function clear(N) {
107107
if (a1 === 2)
108108
bench.end(N);
109109
}
110-
for (var i = 0; i < N; i++) {
110+
for (let i = 0; i < N; i++) {
111111
clearImmediate(setImmediate(cb, 1));
112112
}
113113
setImmediate(cb, 2);

benchmark/timers/timers-breadth-args.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ function main({ n }) {
3232
}
3333

3434
bench.start();
35-
for (var i = 0; i < n; i++) {
35+
for (let i = 0; i < n; i++) {
3636
if (i % 4 === 0)
3737
setTimeout(cb4, 1, 3.14, 1024, true, false);
3838
else if (i % 3 === 0)

benchmark/timers/timers-breadth.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ function main({ n }) {
1313
if (j === n)
1414
bench.end(n);
1515
}
16-
for (var i = 0; i < n; i++) {
16+
for (let i = 0; i < n; i++) {
1717
setTimeout(cb, 1);
1818
}
1919
}

benchmark/timers/timers-cancel-pooled.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -9,15 +9,15 @@ const bench = common.createBenchmark(main, {
99
function main({ n }) {
1010

1111
var timer = setTimeout(() => {}, 1);
12-
for (var i = 0; i < n; i++) {
12+
for (let i = 0; i < n; i++) {
1313
setTimeout(cb, 1);
1414
}
1515
var next = timer._idlePrev;
1616
clearTimeout(timer);
1717

1818
bench.start();
1919

20-
for (var j = 0; j < n; j++) {
20+
for (let j = 0; j < n; j++) {
2121
timer = next;
2222
next = timer._idlePrev;
2323
clearTimeout(timer);

benchmark/timers/timers-cancel-unpooled.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ const bench = common.createBenchmark(main, {
1010
function main({ n, direction }) {
1111

1212
const timersList = [];
13-
for (var i = 0; i < n; i++) {
13+
for (let i = 0; i < n; i++) {
1414
timersList.push(setTimeout(cb, i + 1));
1515
}
1616

benchmark/timers/timers-insert-pooled.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ function main({ n }) {
99

1010
bench.start();
1111

12-
for (var i = 0; i < n; i++) {
12+
for (let i = 0; i < n; i++) {
1313
setTimeout(() => {}, 1);
1414
}
1515

benchmark/timers/timers-insert-unpooled.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ function main({ direction, n }) {
2323
}
2424
bench.end(n);
2525

26-
for (var j = 0; j < n; j++) {
26+
for (let j = 0; j < n; j++) {
2727
clearTimeout(timersList[j]);
2828
}
2929
}

benchmark/timers/timers-timeout-nexttick.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ function main({ n }) {
3030
bench.end(n);
3131
}
3232

33-
for (var i = 0; i < n; i++) {
33+
for (let i = 0; i < n; i++) {
3434
setTimeout(i % 2 ? cb : cb2, 1);
3535
}
3636

benchmark/timers/timers-timeout-pooled.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ function main({ n }) {
2727
bench.end(n);
2828
}
2929

30-
for (var i = 0; i < n; i++) {
30+
for (let i = 0; i < n; i++) {
3131
setTimeout(i % 2 ? cb : cb2, 1);
3232
}
3333

0 commit comments

Comments
 (0)