benchmark: refactor to eliminate redeclared vars

In order to comply with linting rules used in the rest of the code base,
eliminate redeclared variables. A conservative approach is used so as to
avoid unintentional performance issues (for example, as might be seen in
some situations when using `let` instead of `var`).

PR-URL: https://github.com/nodejs/node/pull/5468
Reviewed-By: Brian White <mscdex@mscdex.net>

1 files changed, 3 insertions, 2 deletions

diff --git a/benchmark/querystring/querystring-parse.js b/benchmark/querystring/querystring-parse.js
index 581fe7c216..590b89f307 100644
--- a/benchmark/querystring/querystring-parse.js
+++ b/benchmark/querystring/querystring-parse.js
@@ -43,14 +43,15 @@ function main(conf) {
     querystring.parse(input, '&&&&&&&&&&');
   }
 
+  var i;
   if (type !== 'multicharsep') {
     bench.start();
-    for (var i = 0; i < n; i += 1)
+    for (i = 0; i < n; i += 1)
       querystring.parse(input);
     bench.end(n);
   } else {
     bench.start();
-    for (var i = 0; i < n; i += 1)
+    for (i = 0; i < n; i += 1)
       querystring.parse(input, '&&&&&&&&&&');
     bench.end(n);
   }