You are viewing a plain text version of this content. The canonical link for it is here.
Posted to batik-commits@xmlgraphics.apache.org by dv...@apache.org on 2008/05/14 21:06:41 UTC

svn commit: r656365 - /xmlgraphics/batik/trunk/sources/org/apache/batik/dom/util/DOMUtilities.java

Author: dvholten
Date: Wed May 14 12:06:41 2008
New Revision: 656365

URL: http://svn.apache.org/viewvc?rev=656365&view=rev
Log:
use more efficient iteration

Modified:
    xmlgraphics/batik/trunk/sources/org/apache/batik/dom/util/DOMUtilities.java

Modified: xmlgraphics/batik/trunk/sources/org/apache/batik/dom/util/DOMUtilities.java
URL: http://svn.apache.org/viewvc/xmlgraphics/batik/trunk/sources/org/apache/batik/dom/util/DOMUtilities.java?rev=656365&r1=656364&r2=656365&view=diff
==============================================================================
--- xmlgraphics/batik/trunk/sources/org/apache/batik/dom/util/DOMUtilities.java (original)
+++ xmlgraphics/batik/trunk/sources/org/apache/batik/dom/util/DOMUtilities.java Wed May 14 12:06:41 2008
@@ -392,17 +392,17 @@
             // Copy the prefixes from the prefixes map to the wrapper element
             if (prefixes != null) {
                 wrapperElementPrefix += " ";
-                Set keySet = prefixes.keySet();
-                Iterator iter = keySet.iterator();
+                Iterator iter = prefixes.entrySet().iterator();
                 while (iter.hasNext()) {
-                    String currentKey = (String) iter.next();
-                    String currentValue = (String) prefixes.get(currentKey);
+                    Map.Entry e = (Map.Entry) iter.next();
+                    String currentKey = (String) e.getKey();
+                    String currentValue = (String) e.getValue();
                     wrapperElementPrefix += currentKey + "=\"" + currentValue
                             + "\" ";
                 }
             }
             wrapperElementPrefix += ">";
-            wrapperElementSuffix += "</" + wrapperElementName + ">";
+            wrapperElementSuffix += "</" + wrapperElementName + '>';
         }
 
         // Try and parse as a whole document, if no wrapper element is specified