You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@struts.apache.org by lu...@apache.org on 2011/12/16 18:02:42 UTC

svn commit: r1215224 - /struts/struts2/trunk/core/src/main/java/org/apache/struts2/dispatcher/mapper/DefaultActionMapper.java

Author: lukaszlenart
Date: Fri Dec 16 17:02:42 2011
New Revision: 1215224

URL: http://svn.apache.org/viewvc?rev=1215224&view=rev
Log:
Improves adding tokens to collection

Modified:
    struts/struts2/trunk/core/src/main/java/org/apache/struts2/dispatcher/mapper/DefaultActionMapper.java

Modified: struts/struts2/trunk/core/src/main/java/org/apache/struts2/dispatcher/mapper/DefaultActionMapper.java
URL: http://svn.apache.org/viewvc/struts/struts2/trunk/core/src/main/java/org/apache/struts2/dispatcher/mapper/DefaultActionMapper.java?rev=1215224&r1=1215223&r2=1215224&view=diff
==============================================================================
--- struts/struts2/trunk/core/src/main/java/org/apache/struts2/dispatcher/mapper/DefaultActionMapper.java (original)
+++ struts/struts2/trunk/core/src/main/java/org/apache/struts2/dispatcher/mapper/DefaultActionMapper.java Fri Dec 16 17:02:42 2011
@@ -276,9 +276,7 @@ public class DefaultActionMapper impleme
         if (extensions != null && !"".equals(extensions)) {
             List<String> list = new ArrayList<String>();
             String[] tokens = extensions.split(",");
-            for (String token : tokens) {
-                list.add(token);
-            }
+            Collections.addAll(list, tokens);
             if (extensions.endsWith(",")) {
                 list.add("");
             }
@@ -336,6 +334,7 @@ public class DefaultActionMapper impleme
             int exclamation = name.lastIndexOf("!");
             if (exclamation != -1) {
                 mapping.setName(name.substring(0, exclamation));
+
                 mapping.setMethod(name.substring(exclamation + 1));
             }
         }