tools: enable block-scoped-var eslint rule

PR-URL: https://github.com/nodejs/node/pull/27616
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

1 files changed, 8 insertions, 6 deletions

diff --git a/lib/internal/util/inspect.js b/lib/internal/util/inspect.js
index 21b9921dbe..1e7277ef38 100644
--- a/lib/internal/util/inspect.js
+++ b/lib/internal/util/inspect.js
@@ -312,7 +312,8 @@ function strEscape(str) {
 
   let result = '';
   let last = 0;
-  for (var i = 0; i < str.length; i++) {
+  const lastIndex = str.length;
+  for (let i = 0; i < lastIndex; i++) {
     const point = str.charCodeAt(i);
     if (point === singleQuote || point === 92 || point < 32) {
       if (last === i) {
@@ -324,7 +325,7 @@ function strEscape(str) {
     }
   }
 
-  if (last !== i) {
+  if (last !== lastIndex) {
     result += str.slice(last);
   }
   return addQuotes(result, singleQuote);
@@ -1061,10 +1062,11 @@ function formatPrimitive(fn, value, ctx) {
       if (matches.length > 1) {
         const indent = ' '.repeat(ctx.indentationLvl);
         let res = `${fn(strEscape(matches[0]), 'string')} +\n`;
-        for (var i = 1; i < matches.length - 1; i++) {
+        const lastIndex = matches.length - 1;
+        for (let i = 1; i < lastIndex; i++) {
           res += `${indent}  ${fn(strEscape(matches[i]), 'string')} +\n`;
         }
-        res += `${indent}  ${fn(strEscape(matches[i]), 'string')}`;
+        res += `${indent}  ${fn(strEscape(matches[lastIndex]), 'string')}`;
         return res;
       }
     }
@@ -1187,10 +1189,10 @@ function formatTypedArray(ctx, value, recurseTimes) {
   const elementFormatter = value.length > 0 && typeof value[0] === 'number' ?
     formatNumber :
     formatBigInt;
-  for (var i = 0; i < maxLength; ++i)
+  for (let i = 0; i < maxLength; ++i)
     output[i] = elementFormatter(ctx.stylize, value[i]);
   if (remaining > 0) {
-    output[i] = `... ${remaining} more item${remaining > 1 ? 's' : ''}`;
+    output[maxLength] = `... ${remaining} more item${remaining > 1 ? 's' : ''}`;
   }
   if (ctx.showHidden) {
     // .buffer goes last, it's not a primitive like the others.