You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by re...@apache.org on 2009/03/30 12:55:13 UTC

svn commit: r759914 - /cocoon/cocoon3/trunk/cocoon-sitemap/src/main/java/org/apache/cocoon/sitemap/matcher/WildcardMatcher.java

Author: reinhard
Date: Mon Mar 30 10:55:13 2009
New Revision: 759914

URL: http://svn.apache.org/viewvc?rev=759914&view=rev
Log:
remove unnecessary if statement. thanks to Dariusz Ɓuksza

Modified:
    cocoon/cocoon3/trunk/cocoon-sitemap/src/main/java/org/apache/cocoon/sitemap/matcher/WildcardMatcher.java

Modified: cocoon/cocoon3/trunk/cocoon-sitemap/src/main/java/org/apache/cocoon/sitemap/matcher/WildcardMatcher.java
URL: http://svn.apache.org/viewvc/cocoon/cocoon3/trunk/cocoon-sitemap/src/main/java/org/apache/cocoon/sitemap/matcher/WildcardMatcher.java?rev=759914&r1=759913&r2=759914&view=diff
==============================================================================
--- cocoon/cocoon3/trunk/cocoon-sitemap/src/main/java/org/apache/cocoon/sitemap/matcher/WildcardMatcher.java (original)
+++ cocoon/cocoon3/trunk/cocoon-sitemap/src/main/java/org/apache/cocoon/sitemap/matcher/WildcardMatcher.java Mon Mar 30 10:55:13 2009
@@ -38,14 +38,12 @@
         }
 
         Map<String, String> result = new HashMap<String, String>();
-        if (matchingResult != null) {
-            for (String key : matchingResult.keySet()) {
-                String value = matchingResult.get(key);
-                result.put(key, value);
-                String param = parameterGroups.get(key);
-                if (param != null) {
-                    result.put(param, value);
-                }
+        for (String key : matchingResult.keySet()) {
+            String value = matchingResult.get(key);
+            result.put(key, value);
+            String param = parameterGroups.get(key);
+            if (param != null) {
+                result.put(param, value);
             }
         }