terser 5.18.1 → 5.18.2

package/CHANGELOG.md

@@ -1,9 +1,12 @@
 # Changelog
 
-## v7.18.1
+## v5.18.2
+ - Stop using recursion in hoisted defuns fix.
+
+## v5.18.1
  - Fix major performance issue caused by hoisted defuns' scopes bugfix.
 
-## v7.18.0
+## v5.18.0
  - Add new `/*@__MANGLE_PROP__*/` annotation, to mark properties that should be mangled.
 
 ## v5.17.7

package/dist/bundle.min.js

@@ -16013,23 +16013,25 @@ function handle_defined_after_hoist(parent) {
 
             // find the index in `found_symbols`, with some special rules:
             const find = (sym_id, starting_at = 0, must_be_write = false) => {
-                const index = found_symbols.indexOf(sym_id, starting_at);
+                let index = starting_at;
 
-                if (
-                    defun_range
-                    && index >= defun_range.start
-                    && index < defun_range.end
-                ) {
-                    return find(sym_id, defun_range.end, must_be_write);
-                } else if (
-                    must_be_write
-                    && index >= 0
-                    && !found_symbol_writes.has(index)
-                ) {
-                    return find(sym_id, index + 1, must_be_write);
-                } else {
-                    return index;
+                for (;;) {
+                    index = found_symbols.indexOf(sym_id, index);
+
+                    if (index === -1) {
+                        break;
+                    } else if (index >= defun_range.start && index < defun_range.end) {
+                        index = defun_range.end;
+                        continue;
+                    } else if (must_be_write && !found_symbol_writes.has(index)) {
+                        index++;
+                        continue;
+                    } else {
+                        break;
+                    }
                 }
+
+                return index;
             };
 
             const read_defun_at = find(fname_def.id);

package/lib/compress/reduce-vars.js

@@ -592,23 +592,25 @@ function handle_defined_after_hoist(parent) {
 
             // find the index in `found_symbols`, with some special rules:
             const find = (sym_id, starting_at = 0, must_be_write = false) => {
-                const index = found_symbols.indexOf(sym_id, starting_at);
-
-                if (
-                    defun_range
-                    && index >= defun_range.start
-                    && index < defun_range.end
-                ) {
-                    return find(sym_id, defun_range.end, must_be_write);
-                } else if (
-                    must_be_write
-                    && index >= 0
-                    && !found_symbol_writes.has(index)
-                ) {
-                    return find(sym_id, index + 1, must_be_write);
-                } else {
-                    return index;
+                let index = starting_at;
+
+                for (;;) {
+                    index = found_symbols.indexOf(sym_id, index);
+
+                    if (index === -1) {
+                        break;
+                    } else if (index >= defun_range.start && index < defun_range.end) {
+                        index = defun_range.end;
+                        continue;
+                    } else if (must_be_write && !found_symbol_writes.has(index)) {
+                        index++;
+                        continue;
+                    } else {
+                        break;
+                    }
                 }
+
+                return index;
             };
 
             const read_defun_at = find(fname_def.id);

package/package.json

@@ -4,7 +4,7 @@
   "homepage": "https://terser.org",
   "author": "Mihai Bazon <mihai.bazon@gmail.com> (http://lisperator.net/)",
   "license": "BSD-2-Clause",
-  "version": "5.18.1",
+  "version": "5.18.2",
   "engines": {
     "node": ">=10"
   },