Skip to content

Commit 148df0a

Browse files
dnlupaddaleax
authored andcommitted
benchmark: use let instead of var in url
PR-URL: #31794 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]>
1 parent 108e91f commit 148df0a

3 files changed

+10
-10
lines changed

benchmark/url/legacy-vs-whatwg-url-parse.js

+5-5
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ const bench = common.createBenchmark(main, {
1313

1414
function useLegacy(data) {
1515
const len = data.length;
16-
var result = url.parse(data[0]); // Avoid dead code elimination
16+
let result = url.parse(data[0]); // Avoid dead code elimination
1717
bench.start();
1818
for (let i = 0; i < len; ++i) {
1919
result = url.parse(data[i]);
@@ -24,7 +24,7 @@ function useLegacy(data) {
2424

2525
function useWHATWGWithBase(data) {
2626
const len = data.length;
27-
var result = new URL(data[0][0], data[0][1]); // Avoid dead code elimination
27+
let result = new URL(data[0][0], data[0][1]); // Avoid dead code elimination
2828
bench.start();
2929
for (let i = 0; i < len; ++i) {
3030
const item = data[i];
@@ -36,7 +36,7 @@ function useWHATWGWithBase(data) {
3636

3737
function useWHATWGWithoutBase(data) {
3838
const len = data.length;
39-
var result = new URL(data[0]); // Avoid dead code elimination
39+
let result = new URL(data[0]); // Avoid dead code elimination
4040
bench.start();
4141
for (let i = 0; i < len; ++i) {
4242
result = new URL(data[i]);
@@ -47,8 +47,8 @@ function useWHATWGWithoutBase(data) {
4747

4848
function main({ e, method, type, withBase }) {
4949
withBase = withBase === 'true';
50-
var noDead; // Avoid dead code elimination.
51-
var data;
50+
let noDead; // Avoid dead code elimination.
51+
let data;
5252
switch (method) {
5353
case 'legacy':
5454
data = common.bakeUrlData(type, e, false, false);

benchmark/url/legacy-vs-whatwg-url-serialize.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ const bench = common.createBenchmark(main, {
1313
function useLegacy(data) {
1414
const obj = url.parse(data[0]);
1515
const len = data.length;
16-
var noDead = url.format(obj);
16+
let noDead = url.format(obj);
1717
bench.start();
1818
for (let i = 0; i < len; i++) {
1919
noDead = data[i].toString();
@@ -25,7 +25,7 @@ function useLegacy(data) {
2525
function useWHATWG(data) {
2626
const obj = new URL(data[0]);
2727
const len = data.length;
28-
var noDead = obj.toString();
28+
let noDead = obj.toString();
2929
bench.start();
3030
for (let i = 0; i < len; i++) {
3131
noDead = data[i].toString();
@@ -37,7 +37,7 @@ function useWHATWG(data) {
3737
function main({ type, e, method }) {
3838
const data = common.bakeUrlData(type, e, false, false);
3939

40-
var noDead; // Avoid dead code elimination.
40+
let noDead; // Avoid dead code elimination.
4141
switch (method) {
4242
case 'legacy':
4343
noDead = useLegacy(data);

benchmark/url/whatwg-url-properties.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ const bench = common.createBenchmark(main, {
1212

1313
function setAndGet(data, prop) {
1414
const len = data.length;
15-
var result = data[0][prop];
15+
let result = data[0][prop];
1616
bench.start();
1717
for (let i = 0; i < len; ++i) {
1818
result = data[i][prop];
@@ -24,7 +24,7 @@ function setAndGet(data, prop) {
2424

2525
function get(data, prop) {
2626
const len = data.length;
27-
var result = data[0][prop];
27+
let result = data[0][prop];
2828
bench.start();
2929
for (let i = 0; i < len; ++i) {
3030
result = data[i][prop]; // get

0 commit comments

Comments
 (0)