You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by mc...@apache.org on 2009/08/24 17:20:54 UTC

svn commit: r807282 - /myfaces/core/trunk/api/src/main/java/javax/faces/webapp/_ErrorPageWriter.java

Author: mconcini
Date: Mon Aug 24 15:20:52 2009
New Revision: 807282

URL: http://svn.apache.org/viewvc?rev=807282&view=rev
Log:
quick todo to convert regex pattern to a constant

Modified:
    myfaces/core/trunk/api/src/main/java/javax/faces/webapp/_ErrorPageWriter.java

Modified: myfaces/core/trunk/api/src/main/java/javax/faces/webapp/_ErrorPageWriter.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/api/src/main/java/javax/faces/webapp/_ErrorPageWriter.java?rev=807282&r1=807281&r2=807282&view=diff
==============================================================================
--- myfaces/core/trunk/api/src/main/java/javax/faces/webapp/_ErrorPageWriter.java (original)
+++ myfaces/core/trunk/api/src/main/java/javax/faces/webapp/_ErrorPageWriter.java Mon Aug 24 15:20:52 2009
@@ -59,6 +59,8 @@
     private static final String DEBUG_TEMPLATE_RESOURCE = "org.apache.myfaces.DEBUG_TEMPLATE_RESOURCE";
 
     private static String[] DEBUG_PARTS;
+    
+    private static final String REGEX_PATTERN = ".*?\\Q,Id:\\E\\s*(\\S+)\\s*\\].*?";
 
     public _ErrorPageWriter()
     {
@@ -124,8 +126,7 @@
             return null;
 
         List<String> list = new ArrayList<String>();
-        // TODO: Use a constant to reduce the compilation overhaul
-        Pattern pattern = Pattern.compile(".*?\\Q,Id:\\E\\s*(\\S+)\\s*\\].*?");
+        Pattern pattern = Pattern.compile(REGEX_PATTERN);
         Matcher matcher = pattern.matcher(message);
 
         while (matcher.find())