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());
     }
 
     /**

Reply via email to