Improve DCE by code moving out of static block

jridgewell · Feb 24, 2024 · fc2a76a · fc2a76a
1 parent f8850db
commit fc2a76a
Show file tree

Hide file tree

Showing 3 changed files with 252 additions and 250 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -18,6 +18,12 @@ module.exports = {
     'no-constant-condition': 'off',
     'no-unused-labels': 'off',
     'prefer-rest-params': 'off',
+    'prefer-const': [
+      'error',
+      {
+        destructuring: 'all',
+      },
+    ],
   },
   overrides: [
     {

diff --git a/src/by-source.ts b/src/by-source.ts
@@ -34,13 +34,14 @@ export default function buildBySources(
       // segment should go. Either way, we want to insert after that. And there may be multiple
       // generated segments associated with an original location, so there may need to move several
       // indexes before we find where we need to insert.
-      const index = upperBound(
+      let index = upperBound(
         originalLine,
         sourceColumn,
         memoizedBinarySearch(originalLine, sourceColumn, memo, sourceLine),
       );
 
-      insert(originalLine, (memo.lastIndex = index + 1), [sourceColumn, i, seg[COLUMN]]);
+      memo.lastIndex = ++index;
+      insert(originalLine, index, [sourceColumn, i, seg[COLUMN]]);
     }
   }