You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by bo...@apache.org on 2016/12/21 10:01:23 UTC
[2/2] ant git commit: extract assignment statements
extract assignment statements
Project: http://git-wip-us.apache.org/repos/asf/ant/repo
Commit: http://git-wip-us.apache.org/repos/asf/ant/commit/7b9ca06e
Tree: http://git-wip-us.apache.org/repos/asf/ant/tree/7b9ca06e
Diff: http://git-wip-us.apache.org/repos/asf/ant/diff/7b9ca06e
Branch: refs/heads/master
Commit: 7b9ca06ea2e80232546c804639b93f896a3bd7f6
Parents: ef80b11
Author: Stefan Bodewig <bo...@apache.org>
Authored: Wed Dec 21 11:00:48 2016 +0100
Committer: Stefan Bodewig <bo...@apache.org>
Committed: Wed Dec 21 11:00:48 2016 +0100
----------------------------------------------------------------------
src/main/org/apache/tools/ant/filters/UniqFilter.java | 7 +++++--
src/main/org/apache/tools/ant/taskdefs/Sync.java | 3 ++-
.../ant/taskdefs/optional/junit/AggregateTransformer.java | 6 ++++--
src/main/org/apache/tools/bzip2/BlockSort.java | 6 ++++--
4 files changed, 15 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ant/blob/7b9ca06e/src/main/org/apache/tools/ant/filters/UniqFilter.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/filters/UniqFilter.java b/src/main/org/apache/tools/ant/filters/UniqFilter.java
index e72d5f5..2983b6c 100644
--- a/src/main/org/apache/tools/ant/filters/UniqFilter.java
+++ b/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;
}
}
http://git-wip-us.apache.org/repos/asf/ant/blob/7b9ca06e/src/main/org/apache/tools/ant/taskdefs/Sync.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/Sync.java b/src/main/org/apache/tools/ant/taskdefs/Sync.java
index c1c7552..bcdfd52 100644
--- a/src/main/org/apache/tools/ant/taskdefs/Sync.java
+++ b/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);
http://git-wip-us.apache.org/repos/asf/ant/blob/7b9ca06e/src/main/org/apache/tools/ant/taskdefs/optional/junit/AggregateTransformer.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junit/AggregateTransformer.java b/src/main/org/apache/tools/ant/taskdefs/optional/junit/AggregateTransformer.java
index 32dd704..ca47074 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/junit/AggregateTransformer.java
+++ b/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;
}
/**
http://git-wip-us.apache.org/repos/asf/ant/blob/7b9ca06e/src/main/org/apache/tools/bzip2/BlockSort.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/bzip2/BlockSort.java b/src/main/org/apache/tools/bzip2/BlockSort.java
index 78ea15e..b382470 100644
--- a/src/main/org/apache/tools/bzip2/BlockSort.java
+++ b/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;
}
/*