<parallel>: failonany was tested even if the task succeeded.

PR: 28122
Reported by: Oliver Dungey


git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@276270 13f79535-47bb-0310-9956-ffa450edef68

WHATSNEW

@@ -97,6 +97,8 @@ Fixed bugs:
 
 * wrong compare used in ProjectComponent for logging. Bugzilla Report 28070.
 
+* failOnAny attribute for <parallel> was broken. Bugzilla Report 28122.
+
 Other changes:
 --------------
 

src/main/org/apache/tools/ant/taskdefs/Parallel.java

@@ -417,12 +417,12 @@ public class Parallel extends Task
                 task.perform();
             } catch (Throwable t) {
                 exception = t;
+                if (failOnAny) {
+                    stillRunning = false;
+                }
             } finally {
                 synchronized (semaphore) {
                     finished = true;
-                    if (failOnAny) {
-                        stillRunning = false;
-                    }
                     semaphore.notifyAll();
                 }
             }