benchmark: use let instead of var in async_hooks

dnlup · targos · commit 2ad7003f8d31 · 2019-12-01T17:02:49.000+01:00
PR-URL: #30470 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/benchmark/async_hooks/gc-tracking.js b/benchmark/async_hooks/gc-tracking.js
@@ -22,18 +22,17 @@ function endAfterGC(n) {
 }
 
 function main({ n, method }) {
- var i;
 switch (method) {
 case 'trackingEnabled':
 bench.start();
- for (i = 0; i < n; i++) {
+ for (let i = 0; i < n; i++) {
 new AsyncResource('foobar');
 }
 endAfterGC(n);
 break;
 case 'trackingDisabled':
 bench.start();
- for (i = 0; i < n; i++) {
+ for (let i = 0; i < n; i++) {
 new AsyncResource('foobar', { requireManualDestroy: true });
 }
 endAfterGC(n);

-Original file line number
+Diff line change
+}
 functionmain({ n, method }){
 -vari;
 switch(method){
 case'trackingEnabled':
 bench.start();
 -for(i=0;i<n;i++){
 +for(leti=0;i<n;i++){
 newAsyncResource('foobar');
+}
 endAfterGC(n);
 break;
 case'trackingDisabled':
 bench.start();
 -for(i=0;i<n;i++){
 +for(leti=0;i<n;i++){
 newAsyncResource('foobar',{requireManualDestroy: true});
+}
 endAfterGC(n);