Author: rdonkin
Date: Fri Jul 24 14:34:55 2009
New Revision: 797490

URL: http://svn.apache.org/viewvc?rev=797490&view=rev
Log:
JSIEVE-53 Generify https://issues.apache.org/jira/browse/JSIEVE-53

Modified:
    james/jsieve/trunk/main/src/main/java/org/apache/jsieve/tests/Not.java

Modified: james/jsieve/trunk/main/src/main/java/org/apache/jsieve/tests/Not.java
URL: 
http://svn.apache.org/viewvc/james/jsieve/trunk/main/src/main/java/org/apache/jsieve/tests/Not.java?rev=797490&r1=797489&r2=797490&view=diff
==============================================================================
--- james/jsieve/trunk/main/src/main/java/org/apache/jsieve/tests/Not.java 
(original)
+++ james/jsieve/trunk/main/src/main/java/org/apache/jsieve/tests/Not.java Fri 
Jul 24 14:34:55 2009
@@ -19,7 +19,6 @@
 
 package org.apache.jsieve.tests;
 
-import java.util.Iterator;
 import java.util.List;
 
 import org.apache.jsieve.Arguments;
@@ -47,15 +46,12 @@
     protected boolean executeBasic(MailAdapter mail, Arguments arguments,
             SieveContext context) throws SieveException {
         boolean result = true;
-        List tests = arguments.getTestList().getTests();
+        List<Test> tests = arguments.getTestList().getTests();
         if (tests.size() != 1)
             throw context.getCoordinate().syntaxException(
                     "Exactly 1 test permitted. Found " + tests.size());
-        Iterator testsIter = tests.iterator();
-        while (testsIter.hasNext()) {
-            result = result
-                    && ((Boolean) ((Test) testsIter.next()).execute(mail,
-                            context)).booleanValue();
+        for (Test test: tests) {
+            result = result && test.isTestPassed(mail, context);
         }
         return !result;
     }



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to