terser 5.19.1 → 5.19.2

package/CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## v5.19.2
+ - fix performance hit from avoiding HTML comments in the output
+
 ## v5.19.1
  - Better avoid outputting `</script>` and HTML comments.
  - Fix unused variables in class static blocks not being dropped correctly.

package/dist/bundle.min.js

@@ -8954,18 +8954,6 @@ class Rope {
         this.current += str;
     }
 
-    endsWith(str) {
-        const { committed, current } = this;
-        const len = str.length;
-        if (committed.length >= len) {
-            return committed.slice(committed.length - str.length) === str;
-        } else {
-            // `str` is small and this is a rare case so keep it simple
-            const last_bit = committed.slice(-len) + current;
-            return last_bit.endsWith(str);
-        }
-    }
-
     insertAt(char, index) {
         const { committed, current } = this;
         if (index < committed.length) {
@@ -9347,10 +9335,6 @@ function OutputStream(options) {
         might_need_semicolon = true;
     };
 
-    function ends_with(str) {
-        return OUTPUT.endsWith(str);
-    }
-
     function force_semicolon() {
         might_need_semicolon = false;
         print(";");
@@ -9620,7 +9604,6 @@ function OutputStream(options) {
         comma           : comma,
         colon           : colon,
         last            : function() { return last; },
-        ends_with       : ends_with,
         semicolon       : semicolon,
         force_semicolon : force_semicolon,
         to_utf8         : to_utf8,
@@ -10744,7 +10727,8 @@ function OutputStream(options) {
     });
     DEFPRINT(AST_UnaryPrefix, function(self, output) {
         var op = self.operator;
-        if (op === "--" && output.ends_with("<!")) {
+        if (op === "--" && output.last().endsWith("!")) {
+            // avoid printing "<!--"
             output.print(" ");
         }
         output.print(op);
@@ -10764,7 +10748,7 @@ function OutputStream(options) {
         var op = self.operator;
         self.left.print(output);
         if (op[0] == ">" /* ">>" ">>>" ">" ">=" */
-            && output.ends_with("--")) {
+            && output.last().endsWith("--")) {
             // space is mandatory to avoid outputting -->
             output.print(" ");
         } else {
@@ -11083,7 +11067,7 @@ function OutputStream(options) {
 
         // Avoid outputting end of script tag
         source = source.replace(r_slash_script, slash_script_replace);
-        if (r_starts_with_script.test(source) && output.ends_with("<")) {
+        if (r_starts_with_script.test(source) && output.last().endsWith("<")) {
             output.print(" ");
         }
 

package/lib/output.js

@@ -190,18 +190,6 @@ class Rope {
         this.current += str;
     }
 
-    endsWith(str) {
-        const { committed, current } = this;
-        const len = str.length;
-        if (committed.length >= len) {
-            return committed.slice(committed.length - str.length) === str;
-        } else {
-            // `str` is small and this is a rare case so keep it simple
-            const last_bit = committed.slice(-len) + current;
-            return last_bit.endsWith(str);
-        }
-    }
-
     insertAt(char, index) {
         const { committed, current } = this;
         if (index < committed.length) {
@@ -583,10 +571,6 @@ function OutputStream(options) {
         might_need_semicolon = true;
     };
 
-    function ends_with(str) {
-        return OUTPUT.endsWith(str);
-    }
-
     function force_semicolon() {
         might_need_semicolon = false;
         print(";");
@@ -856,7 +840,6 @@ function OutputStream(options) {
         comma           : comma,
         colon           : colon,
         last            : function() { return last; },
-        ends_with       : ends_with,
         semicolon       : semicolon,
         force_semicolon : force_semicolon,
         to_utf8         : to_utf8,
@@ -1980,7 +1963,8 @@ function OutputStream(options) {
     });
     DEFPRINT(AST_UnaryPrefix, function(self, output) {
         var op = self.operator;
-        if (op === "--" && output.ends_with("<!")) {
+        if (op === "--" && output.last().endsWith("!")) {
+            // avoid printing "<!--"
             output.print(" ");
         }
         output.print(op);
@@ -2000,7 +1984,7 @@ function OutputStream(options) {
         var op = self.operator;
         self.left.print(output);
         if (op[0] == ">" /* ">>" ">>>" ">" ">=" */
-            && output.ends_with("--")) {
+            && output.last().endsWith("--")) {
             // space is mandatory to avoid outputting -->
             output.print(" ");
         } else {
@@ -2319,7 +2303,7 @@ function OutputStream(options) {
 
         // Avoid outputting end of script tag
         source = source.replace(r_slash_script, slash_script_replace);
-        if (r_starts_with_script.test(source) && output.ends_with("<")) {
+        if (r_starts_with_script.test(source) && output.last().endsWith("<")) {
             output.print(" ");
         }
 

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.19.1",
+  "version": "5.19.2",
   "engines": {
     "node": ">=10"
   },