diff --git a/.gitignore b/.gitignore index b628f8003..c93004146 100644 --- a/.gitignore +++ b/.gitignore @@ -6,7 +6,7 @@ bin target /build /lib -/site-content +site-content .ekstazi *.class *.iml diff --git a/commons-math-legacy-core/src/main/java/org/apache/commons/math4/legacy/core/MathArrays.java b/commons-math-legacy-core/src/main/java/org/apache/commons/math4/legacy/core/MathArrays.java index 0c77bf2c1..ba7262357 100644 --- a/commons-math-legacy-core/src/main/java/org/apache/commons/math4/legacy/core/MathArrays.java +++ b/commons-math-legacy-core/src/main/java/org/apache/commons/math4/legacy/core/MathArrays.java @@ -617,7 +617,7 @@ public final class MathArrays { */ public static boolean equals(float[] x, float[] y) { if (x == null || y == null) { - return !((x == null) ^ (y == null)); + return (x == null) == (y == null); } if (x.length != y.length) { return false; @@ -643,7 +643,7 @@ public final class MathArrays { */ public static boolean equalsIncludingNaN(float[] x, float[] y) { if (x == null || y == null) { - return !((x == null) ^ (y == null)); + return (x == null) == (y == null); } if (x.length != y.length) { return false; @@ -668,7 +668,7 @@ public final class MathArrays { */ public static boolean equals(double[] x, double[] y) { if (x == null || y == null) { - return !((x == null) ^ (y == null)); + return (x == null) == (y == null); } if (x.length != y.length) { return false; @@ -694,7 +694,7 @@ public final class MathArrays { */ public static boolean equalsIncludingNaN(double[] x, double[] y) { if (x == null || y == null) { - return !((x == null) ^ (y == null)); + return (x == null) == (y == null); } if (x.length != y.length) { return false; diff --git a/commons-math-legacy-core/src/main/java/org/apache/commons/math4/legacy/core/dfp/Dfp.java b/commons-math-legacy-core/src/main/java/org/apache/commons/math4/legacy/core/dfp/Dfp.java index ceffa21ae..41f15c911 100644 --- a/commons-math-legacy-core/src/main/java/org/apache/commons/math4/legacy/core/dfp/Dfp.java +++ b/commons-math-legacy-core/src/main/java/org/apache/commons/math4/legacy/core/dfp/Dfp.java @@ -2351,10 +2351,7 @@ public class Dfp implements RealFieldElement { } // if this is greater than x - boolean up = false; - if (this.lessThan(x)) { - up = true; - } + boolean up = this.lessThan(x); if (compare(this, x) == 0) { return newInstance(x); diff --git a/commons-math-legacy-core/src/main/java/org/apache/commons/math4/legacy/core/dfp/DfpMath.java b/commons-math-legacy-core/src/main/java/org/apache/commons/math4/legacy/core/dfp/DfpMath.java index f831603d5..4503fb02f 100644 --- a/commons-math-legacy-core/src/main/java/org/apache/commons/math4/legacy/core/dfp/DfpMath.java +++ b/commons-math-legacy-core/src/main/java/org/apache/commons/math4/legacy/core/dfp/DfpMath.java @@ -941,11 +941,7 @@ public final class DfpMath { */ public static Dfp acos(Dfp a) { Dfp result; - boolean negative = false; - - if (a.lessThan(a.getZero())) { - negative = true; - } + boolean negative = a.lessThan(a.getZero()); a = Dfp.copySign(a, a.getOne()); // absolute value diff --git a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/analysis/polynomials/PolynomialFunction.java b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/analysis/polynomials/PolynomialFunction.java index dd382dddb..d9884c8a3 100644 --- a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/analysis/polynomials/PolynomialFunction.java +++ b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/analysis/polynomials/PolynomialFunction.java @@ -371,10 +371,7 @@ public class PolynomialFunction implements UnivariateDifferentiableFunction, Ser return false; } PolynomialFunction other = (PolynomialFunction) obj; - if (!Arrays.equals(coefficients, other.coefficients)) { - return false; - } - return true; + return Arrays.equals(coefficients, other.coefficients); } /** diff --git a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/fitting/SimpleCurveFitter.java b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/fitting/SimpleCurveFitter.java index 671150d3b..012c913ef 100644 --- a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/fitting/SimpleCurveFitter.java +++ b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/fitting/SimpleCurveFitter.java @@ -186,11 +186,7 @@ public class SimpleCurveFitter extends AbstractCurveFitter { if (comp != 0) { return comp; } - comp = Double.compare(p1.getWeight(), p2.getWeight()); - if (comp != 0) { - return comp; - } - return 0; + return Double.compare(p1.getWeight(), p2.getWeight()); } }; diff --git a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/ode/events/EventState.java b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/ode/events/EventState.java index 832067abd..eeec951c8 100644 --- a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/ode/events/EventState.java +++ b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/ode/events/EventState.java @@ -365,7 +365,7 @@ public class EventState { // force the sign to its value "just after the event" previousEventTime = t; g0Positive = increasing; - nextAction = handler.eventOccurred(t, y, !(increasing ^ forward)); + nextAction = handler.eventOccurred(t, y, increasing == forward); } else { g0Positive = g0 >= 0; nextAction = EventHandler.Action.CONTINUE; diff --git a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/ode/events/FieldEventState.java b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/ode/events/FieldEventState.java index 3bf39e7a7..e91bacc36 100644 --- a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/ode/events/FieldEventState.java +++ b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/ode/events/FieldEventState.java @@ -301,7 +301,7 @@ public class FieldEventState> { // force the sign to its value "just after the event" previousEventTime = state.getTime(); g0Positive = increasing; - nextAction = handler.eventOccurred(state, !(increasing ^ forward)); + nextAction = handler.eventOccurred(state, increasing == forward); } else { g0Positive = g0.getReal() >= 0; nextAction = Action.CONTINUE; diff --git a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/optim/linear/SolutionCallback.java b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/optim/linear/SolutionCallback.java index 4d2b542cd..cdccdbd2b 100644 --- a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/optim/linear/SolutionCallback.java +++ b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/optim/linear/SolutionCallback.java @@ -57,6 +57,6 @@ public class SolutionCallback implements OptimizationData { * @return {@code true} if the solution is optimal, {@code false} otherwise */ public boolean isSolutionOptimal() { - return tableau != null ? tableau.isOptimal() : false; + return tableau != null && tableau.isOptimal(); } } diff --git a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/optim/nonlinear/scalar/noderiv/CMAESOptimizer.java b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/optim/nonlinear/scalar/noderiv/CMAESOptimizer.java index 38f791927..cfee773f9 100644 --- a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/optim/nonlinear/scalar/noderiv/CMAESOptimizer.java +++ b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/optim/nonlinear/scalar/noderiv/CMAESOptimizer.java @@ -896,7 +896,7 @@ public class CMAESOptimizer @Override public int hashCode() { long bits = Double.doubleToLongBits(value); - return (int) ((1438542 ^ (bits >>> 32) ^ bits) & 0xffffffff); + return (int) ((1438542 ^ (bits >>> 32) ^ bits)); } } /** diff --git a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/Frequency.java b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/Frequency.java index 5b9f51002..e79858bbe 100644 --- a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/Frequency.java +++ b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/Frequency.java @@ -376,14 +376,7 @@ public class Frequency> implements Serializable { return false; } Frequency other = (Frequency) obj; - if (freqTable == null) { - if (other.freqTable != null) { - return false; - } - } else if (!freqTable.equals(other.freqTable)) { - return false; - } - return true; + return freqTable.equals(other.freqTable); } } diff --git a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/descriptive/moment/VectorialCovariance.java b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/descriptive/moment/VectorialCovariance.java index dfa0d716b..c8b5b5aa6 100644 --- a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/descriptive/moment/VectorialCovariance.java +++ b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/descriptive/moment/VectorialCovariance.java @@ -148,10 +148,7 @@ public class VectorialCovariance implements Serializable { if (!Arrays.equals(productsSums, other.productsSums)) { return false; } - if (!Arrays.equals(sums, other.sums)) { - return false; - } - return true; + return Arrays.equals(sums, other.sums); } } diff --git a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/descriptive/moment/VectorialMean.java b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/descriptive/moment/VectorialMean.java index d00b49a78..ade4be441 100644 --- a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/descriptive/moment/VectorialMean.java +++ b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/descriptive/moment/VectorialMean.java @@ -96,10 +96,7 @@ public class VectorialMean implements Serializable { return false; } VectorialMean other = (VectorialMean) obj; - if (!Arrays.equals(means, other.means)) { - return false; - } - return true; + return Arrays.equals(means, other.means); } } diff --git a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/descriptive/rank/PSquarePercentile.java b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/descriptive/rank/PSquarePercentile.java index 603da7614..ed064a996 100644 --- a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/descriptive/rank/PSquarePercentile.java +++ b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/descriptive/rank/PSquarePercentile.java @@ -908,7 +908,7 @@ public class PSquarePercentile extends AbstractStorelessUnivariateStatistic */ @Override public boolean add(final E e) { - return size() < capacity ? super.add(e) : false; + return size() < capacity && super.add(e); } /** @@ -924,7 +924,7 @@ public class PSquarePercentile extends AbstractStorelessUnivariateStatistic boolean isCollectionLess = collection != null && collection.size() + size() <= capacity; - return isCollectionLess ? super.addAll(collection) : false; + return isCollectionLess && super.addAll(collection); } } diff --git a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/inference/KolmogorovSmirnovTest.java b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/inference/KolmogorovSmirnovTest.java index 14ccc1a8b..44cb2c607 100644 --- a/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/inference/KolmogorovSmirnovTest.java +++ b/commons-math-legacy/src/main/java/org/apache/commons/math4/legacy/stat/inference/KolmogorovSmirnovTest.java @@ -1132,11 +1132,7 @@ public class KolmogorovSmirnovTest { throw new InsufficientDataException(); } - if (values.length == x.length + y.length) { - return false; // There are no ties. - } - - return true; + return values.length != x.length + y.length; // There are no ties. } /** diff --git a/src/changes/changes.xml b/src/changes/changes.xml index 282151296..04714991c 100644 --- a/src/changes/changes.xml +++ b/src/changes/changes.xml @@ -45,7 +45,9 @@ with a new-line in the release notes. (These spaces are ignored when displaying If the output is not quite correct, check for invisible trailing spaces! --> - + Apache Commons Math Release Notes @@ -95,6 +97,9 @@ Caveat: nightmare was one of the main reasons for creating more focused components.] "> + + Simplify conditional expressions. + Refactor "EmpiricalDistribution" class (in package "o.a.c.m.legacy.distribution").