Author: rdonkin
Date: Fri Jul 24 14:24:49 2009
New Revision: 797480
URL: http://svn.apache.org/viewvc?rev=797480&view=rev
Log:
JSIEVE-53 Reuse call rather than duplicate method
https://issues.apache.org/jira/browse/JSIEVE-53
Modified:
james/jsieve/trunk/main/src/main/java/org/apache/jsieve/tests/AllOf.java
Modified:
james/jsieve/trunk/main/src/main/java/org/apache/jsieve/tests/AllOf.java
URL:
http://svn.apache.org/viewvc/james/jsieve/trunk/main/src/main/java/org/apache/jsieve/tests/AllOf.java?rev=797480&r1=797479&r2=797480&view=diff
==============================================================================
--- james/jsieve/trunk/main/src/main/java/org/apache/jsieve/tests/AllOf.java
(original)
+++ james/jsieve/trunk/main/src/main/java/org/apache/jsieve/tests/AllOf.java
Fri Jul 24 14:24:49 2009
@@ -19,11 +19,8 @@
package org.apache.jsieve.tests;
-import java.util.Iterator;
-
import org.apache.jsieve.Arguments;
import org.apache.jsieve.SieveContext;
-import org.apache.jsieve.Test;
import org.apache.jsieve.exception.SieveException;
import org.apache.jsieve.mail.MailAdapter;
@@ -45,13 +42,7 @@
*/
protected boolean executeBasic(MailAdapter mail, Arguments arguments,
SieveContext context) throws SieveException {
- boolean result = true;
- Iterator testsIter = arguments.getTestList().getTests().iterator();
- while (result && testsIter.hasNext()) {
- result = ((Boolean) ((Test) testsIter.next())
- .execute(mail, context)).booleanValue();
- }
- return result;
+ return arguments.getTestList().isTestPassed(mail, context);
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
