Minor performance optimization

* Minor performance optimization for at rule check.
puckowski · Jul 21, 2023 · b649e5c · b649e5c
1 parent 7a4d9a8
commit b649e5c
Show file tree

Hide file tree

Showing 8 changed files with 45 additions and 20 deletions.
diff --git a/dist/less.js b/dist/less.js
@@ -2952,6 +2952,12 @@
             skipWhitespace(1);
             return tok;
         };
+        parserInput.$peekChar = function (tok) {
+            if (input.charAt(parserInput.i) !== tok) {
+                return null;
+            }
+            return tok;
+        };
         parserInput.$str = function (tok) {
             var tokLength = tok.length;
             // https://jsperf.com/string-startswith/21
@@ -4910,11 +4916,13 @@
                         debugInfo = getDebugInfo(index);
                     }
                     parserInput.save();
-                    if (parserInput.$str('@media')) {
-                        return this.prepareAndGetNestableAtRule(tree.Media, index, debugInfo, MediaSyntaxOptions);
-                    }
-                    if (parserInput.$str('@container')) {
-                        return this.prepareAndGetNestableAtRule(tree.Container, index, debugInfo, ContainerSyntaxOptions);
+                    if (parserInput.$peekChar('@')) {
+                        if (parserInput.$str('@media')) {
+                            return this.prepareAndGetNestableAtRule(tree.Media, index, debugInfo, MediaSyntaxOptions);
+                        }
+                        if (parserInput.$str('@container')) {
+                            return this.prepareAndGetNestableAtRule(tree.Container, index, debugInfo, ContainerSyntaxOptions);
+                        }
                     }
                     parserInput.restore();
                 },

diff --git a/dist/less.min.js b/dist/less.min.js
diff --git a/dist/less.min.js.map b/dist/less.min.js.map
diff --git a/packages/less/dist/less.js b/packages/less/dist/less.js
@@ -2952,6 +2952,12 @@
             skipWhitespace(1);
             return tok;
         };
+        parserInput.$peekChar = function (tok) {
+            if (input.charAt(parserInput.i) !== tok) {
+                return null;
+            }
+            return tok;
+        };
         parserInput.$str = function (tok) {
             var tokLength = tok.length;
             // https://jsperf.com/string-startswith/21
@@ -4910,11 +4916,13 @@
                         debugInfo = getDebugInfo(index);
                     }
                     parserInput.save();
-                    if (parserInput.$str('@media')) {
-                        return this.prepareAndGetNestableAtRule(tree.Media, index, debugInfo, MediaSyntaxOptions);
-                    }
-                    if (parserInput.$str('@container')) {
-                        return this.prepareAndGetNestableAtRule(tree.Container, index, debugInfo, ContainerSyntaxOptions);
+                    if (parserInput.$peekChar('@')) {
+                        if (parserInput.$str('@media')) {
+                            return this.prepareAndGetNestableAtRule(tree.Media, index, debugInfo, MediaSyntaxOptions);
+                        }
+                        if (parserInput.$str('@container')) {
+                            return this.prepareAndGetNestableAtRule(tree.Container, index, debugInfo, ContainerSyntaxOptions);
+                        }
                     }
                     parserInput.restore();
                 },

diff --git a/packages/less/dist/less.min.js b/packages/less/dist/less.min.js
diff --git a/packages/less/dist/less.min.js.map b/packages/less/dist/less.min.js.map
diff --git a/packages/less/src/less/parser/parser-input.js b/packages/less/src/less/parser/parser-input.js
@@ -149,6 +149,13 @@ export default () => {
         return tok;
     };
 
+    parserInput.$peekChar = tok => {
+        if (input.charAt(parserInput.i) !== tok) {
+            return null;
+        }
+        return tok;
+    };
+
     parserInput.$str = tok => {
         const tokLength = tok.length;
 

diff --git a/packages/less/src/less/parser/parser.js b/packages/less/src/less/parser/parser.js
@@ -1845,12 +1845,14 @@ const Parser = function Parser(context, imports, fileInfo, currentIndex) {
                 }
                 parserInput.save();
 
-                if (parserInput.$str('@media')) {
-                    return this.prepareAndGetNestableAtRule(tree.Media, index, debugInfo, MediaSyntaxOptions);
-                }
-
-                if (parserInput.$str('@container')) {
-                    return this.prepareAndGetNestableAtRule(tree.Container, index, debugInfo, ContainerSyntaxOptions);
+                if (parserInput.$peekChar('@')) {
+                    if (parserInput.$str('@media')) {
+                        return this.prepareAndGetNestableAtRule(tree.Media, index, debugInfo, MediaSyntaxOptions);
+                    }
+
+                    if (parserInput.$str('@container')) {
+                        return this.prepareAndGetNestableAtRule(tree.Container, index, debugInfo, ContainerSyntaxOptions);
+                    }
                 }
 
                 parserInput.restore();