future#isDone() potentially needs synchronized access (expensive), therefore...

future#isDone() potentially needs synchronized access (expensive), therefore use it as 2nd clause in the AND expression

maven-plugin/src/main/java/hudson/maven/AbstractMavenBuilder.java

@@ -139,7 +139,7 @@ public abstract class AbstractMavenBuilder implements DelegatingCallable<Result,
             
             for (Future<?> f : futures) {
                 try {
-                    if(!f.isDone() && !messageReported) {
+                    if(!messageReported && !f.isDone()) {
                         messageReported = true;
                         listener.getLogger().println(Messages.MavenBuilder_Waiting());
                     }