extract assignment statements

src/main/org/apache/tools/ant/filters/UniqFilter.java

@@ -31,7 +31,10 @@ public class UniqFilter extends TokenFilter.ChainableReaderFilter {
     private String lastLine = null;
 
     public String filter(String string) {
-        return lastLine == null || !lastLine.equals(string)
-            ? (lastLine = string) : null;
+        if (lastLine == null || !lastLine.equals(string)) {
+            lastLine = string;
+            return lastLine;
+        }
+        return null;
     }
 }

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

@@ -403,7 +403,8 @@ public class Sync extends Task {
             if (resources == null) {
                 Restrict r = new Restrict();
                 r.add(new Exists());
-                r.add(resources = new Resources());
+                resources = new Resources();
+                r.add(resources);
                 myCopy.add(r);
             }
             resources.add(rc);

src/main/org/apache/tools/ant/taskdefs/optional/junit/AggregateTransformer.java

@@ -236,8 +236,10 @@ public class AggregateTransformer {
      * @since Ant 1.9.5
      */
     public XSLTProcess.Factory createFactory() {
-        return xsltFactory != null ? xsltFactory
-            : (xsltFactory = xsltTask.createFactory());
+        if (xsltFactory == null) {
+            xsltFactory = xsltTask.createFactory();
+        }
+        return xsltFactory;
     }
 
     /**

src/main/org/apache/tools/bzip2/BlockSort.java

@@ -445,8 +445,10 @@ class BlockSort {
     private int[] eclass;
 
     private int[] getEclass() {
-        return eclass == null
-            ? (eclass = new int[quadrant.length / 2]) : eclass;
+        if (eclass == null) {
+            eclass = new int[quadrant.length / 2];
+        }
+        return eclass;
     }
 
     /*