You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-commits@axis.apache.org by ve...@apache.org on 2017/01/10 22:09:27 UTC

svn commit: r1778214 - /axis/axis2/java/rampart/trunk/modules/rampart-core/src/main/java/org/apache/rampart/policy/SupportingPolicyData.java

Author: veithen
Date: Tue Jan 10 22:09:26 2017
New Revision: 1778214

URL: http://svn.apache.org/viewvc?rev=1778214&view=rev
Log:
Use for loops instead of iterators.

Modified:
    axis/axis2/java/rampart/trunk/modules/rampart-core/src/main/java/org/apache/rampart/policy/SupportingPolicyData.java

Modified: axis/axis2/java/rampart/trunk/modules/rampart-core/src/main/java/org/apache/rampart/policy/SupportingPolicyData.java
URL: http://svn.apache.org/viewvc/axis/axis2/java/rampart/trunk/modules/rampart-core/src/main/java/org/apache/rampart/policy/SupportingPolicyData.java?rev=1778214&r1=1778213&r2=1778214&view=diff
==============================================================================
--- axis/axis2/java/rampart/trunk/modules/rampart-core/src/main/java/org/apache/rampart/policy/SupportingPolicyData.java (original)
+++ axis/axis2/java/rampart/trunk/modules/rampart-core/src/main/java/org/apache/rampart/policy/SupportingPolicyData.java Tue Jan 10 22:09:26 2017
@@ -19,8 +19,6 @@
 
 package org.apache.rampart.policy;
 
-import java.util.Iterator;
-
 import org.apache.ws.secpolicy.model.Header;
 import org.apache.ws.secpolicy.model.SupportingToken;
 
@@ -29,39 +27,31 @@ public class SupportingPolicyData extend
     public void build(SupportingToken token) {
 
         if (token.getSignedParts() != null && !token.getSignedParts().isOptional()) {
-            Iterator<Header> it = token.getSignedParts().getHeaders().iterator();
             this.setSignBody(token.getSignedParts().isBody());
-            while (it.hasNext()) {
-                Header header = it.next();
+            for (Header header : token.getSignedParts().getHeaders()) {
                 this.addSignedPart(header.getNamespace(), header.getName());
             }
         }
 
         if (token.getEncryptedParts() != null && !token.getEncryptedParts().isOptional()) {
-            Iterator<Header> it = token.getEncryptedParts().getHeaders().iterator();
             this.setEncryptBody(token.getEncryptedParts().isBody());
-            while (it.hasNext()) {
-                Header header = it.next();
+            for (Header header : token.getEncryptedParts().getHeaders()) {
                 this.setEncryptedParts(header.getNamespace(), header.getName(),
                         "Header");
             }
         }
 
         if (token.getSignedElements() != null && !token.getSignedElements().isOptional()) {
-            Iterator<String> it = token.getSignedElements().getXPathExpressions()
-                    .iterator();
-            while (it.hasNext()) {
-                this.setSignedElements(it.next());
+            for (String xpath : token.getSignedElements().getXPathExpressions()) {
+                this.setSignedElements(xpath);
             }
             this.addDeclaredNamespaces(token.getSignedElements()
                     .getDeclaredNamespaces());
         }
 
         if (token.getEncryptedElements() != null && !token.getEncryptedElements().isOptional()) {
-            Iterator<String> it = token.getEncryptedElements().getXPathExpressions()
-                    .iterator();
-            while (it.hasNext()) {
-                this.setEncryptedElements(it.next());
+            for (String xpath : token.getEncryptedElements().getXPathExpressions()) {
+                this.setEncryptedElements(xpath);
             }
             if (token.getSignedElements() == null) {
                 this.addDeclaredNamespaces(token.getEncryptedElements()