This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-math.git
The following commit(s) were added to refs/heads/master by this push: new 8f949ae Use isEmpty(). new b7dd3fd Merge branch 'master' of https://gitbox.apache.org/repos/asf/commons-math.git 8f949ae is described below commit 8f949ae86a65b2c0999dfa83b81a2cb7be4c1145 Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Sat Jan 16 21:37:02 2021 -0500 Use isEmpty(). --- .../org/apache/commons/math4/ode/ContinuousOutputFieldModel.java | 6 +++--- .../java/org/apache/commons/math4/ode/ContinuousOutputModel.java | 6 +++--- .../java/org/apache/commons/math4/optim/linear/SimplexSolver.java | 2 +- .../java/org/apache/commons/math4/stat/ranking/NaturalRanking.java | 2 +- .../apache/commons/math4/fitting/WeightedObservedPointsTest.java | 4 ++-- 5 files changed, 10 insertions(+), 10 deletions(-) diff --git a/src/main/java/org/apache/commons/math4/ode/ContinuousOutputFieldModel.java b/src/main/java/org/apache/commons/math4/ode/ContinuousOutputFieldModel.java index c2caecd..9863e32 100644 --- a/src/main/java/org/apache/commons/math4/ode/ContinuousOutputFieldModel.java +++ b/src/main/java/org/apache/commons/math4/ode/ContinuousOutputFieldModel.java @@ -120,11 +120,11 @@ public class ContinuousOutputFieldModel<T extends RealFieldElement<T>> public void append(final ContinuousOutputFieldModel<T> model) throws MathIllegalArgumentException, MaxCountExceededException { - if (model.steps.size() == 0) { + if (model.steps.isEmpty()) { return; } - if (steps.size() == 0) { + if (steps.isEmpty()) { initialTime = model.initialTime; forward = model.forward; } else { @@ -195,7 +195,7 @@ public class ContinuousOutputFieldModel<T extends RealFieldElement<T>> public void handleStep(final FieldStepInterpolator<T> interpolator, final boolean isLast) throws MaxCountExceededException { - if (steps.size() == 0) { + if (steps.isEmpty()) { initialTime = interpolator.getPreviousState().getTime(); forward = interpolator.isForward(); } diff --git a/src/main/java/org/apache/commons/math4/ode/ContinuousOutputModel.java b/src/main/java/org/apache/commons/math4/ode/ContinuousOutputModel.java index d89b700..c3589a8 100644 --- a/src/main/java/org/apache/commons/math4/ode/ContinuousOutputModel.java +++ b/src/main/java/org/apache/commons/math4/ode/ContinuousOutputModel.java @@ -129,11 +129,11 @@ public class ContinuousOutputModel public void append(final ContinuousOutputModel model) throws MathIllegalArgumentException, MaxCountExceededException { - if (model.steps.size() == 0) { + if (model.steps.isEmpty()) { return; } - if (steps.size() == 0) { + if (steps.isEmpty()) { initialTime = model.initialTime; forward = model.forward; } else { @@ -190,7 +190,7 @@ public class ContinuousOutputModel public void handleStep(final StepInterpolator interpolator, final boolean isLast) throws MaxCountExceededException { - if (steps.size() == 0) { + if (steps.isEmpty()) { initialTime = interpolator.getPreviousTime(); forward = interpolator.isForward(); } diff --git a/src/main/java/org/apache/commons/math4/optim/linear/SimplexSolver.java b/src/main/java/org/apache/commons/math4/optim/linear/SimplexSolver.java index ae8eb8c..d762bd8 100644 --- a/src/main/java/org/apache/commons/math4/optim/linear/SimplexSolver.java +++ b/src/main/java/org/apache/commons/math4/optim/linear/SimplexSolver.java @@ -270,7 +270,7 @@ public class SimplexSolver extends LinearOptimizer { } } - if (minRatioPositions.size() == 0) { + if (minRatioPositions.isEmpty()) { return null; } else if (minRatioPositions.size() > 1) { // there's a degeneracy as indicated by a tie in the minimum ratio test diff --git a/src/main/java/org/apache/commons/math4/stat/ranking/NaturalRanking.java b/src/main/java/org/apache/commons/math4/stat/ranking/NaturalRanking.java index 4d55867..cb07ad5 100644 --- a/src/main/java/org/apache/commons/math4/stat/ranking/NaturalRanking.java +++ b/src/main/java/org/apache/commons/math4/stat/ranking/NaturalRanking.java @@ -395,7 +395,7 @@ public class NaturalRanking implements RankingAlgorithm { * @param nanPositions list of index values to set to <code>Double.NaN</code> */ private void restoreNaNs(double[] ranks, List<Integer> nanPositions) { - if (nanPositions.size() == 0) { + if (nanPositions.isEmpty()) { return; } Iterator<Integer> iterator = nanPositions.iterator(); diff --git a/src/test/java/org/apache/commons/math4/fitting/WeightedObservedPointsTest.java b/src/test/java/org/apache/commons/math4/fitting/WeightedObservedPointsTest.java index ea0e9c8..186d0e5 100644 --- a/src/test/java/org/apache/commons/math4/fitting/WeightedObservedPointsTest.java +++ b/src/test/java/org/apache/commons/math4/fitting/WeightedObservedPointsTest.java @@ -74,7 +74,7 @@ public class WeightedObservedPointsTest { Assert.assertTrue(store.toList().size() == 2); store.clear(); - Assert.assertTrue(store.toList().size() == 0); + Assert.assertTrue(store.toList().isEmpty()); } // Ensure that an instance returned by "toList()" is independent from @@ -95,7 +95,7 @@ public class WeightedObservedPointsTest { // Clearing "store" has no impact on "list". store.clear(); - Assert.assertFalse(list.size() == 0); + Assert.assertFalse(list.isEmpty()); } /**