Merge pull request #19 from davydotcom/patch-1

Regex patterns are heavy to iteratively compile.
grails · May 8, 2018 · 4110d67 · 4110d67
2 parents eb29930 + 62c12ab
commit 4110d67
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/grails-gsp/src/main/groovy/org/grails/gsp/compiler/GroovyPageScanner.java b/grails-gsp/src/main/groovy/org/grails/gsp/compiler/GroovyPageScanner.java
@@ -19,6 +19,7 @@
 
 import java.util.ArrayList;
 import java.util.List;
+import java.util.regex.Pattern;
 
 /**
  * NOTE: Based on work done by on the GSP standalone project (https://gsp.dev.java.net/)
@@ -44,6 +45,7 @@ class GroovyPageScanner implements Tokens {
     private int lastLineNumberIndex = -1;
     private String pageName = "Unknown";
     private int maxHtmlLength = DEFAULT_MAX_HTML_LENGTH;
+    private Pattern isTagNameSpacePattern = Pattern.compile("^\\p{Alpha}\\w*$");
 
     GroovyPageScanner(String text) {
         Strip strip = new Strip(text);
@@ -247,7 +249,7 @@ private boolean isClosingTag(char c1) {
     }
 
     private boolean isTagDefinition(String tagNameSpace) {
-        return tagNameSpace != null && tagNameSpace.matches("^\\p{Alpha}\\w*$");
+        return tagNameSpace != null && isTagNameSpacePattern.matcher(tagNameSpace).matches();
     }
 
     private String getTagNamespace(int fromIndex) {