You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by lu...@apache.org on 2007/09/10 10:22:54 UTC

svn commit: r574168 - /commons/proper/math/trunk/src/java/org/apache/commons/math/optimization/DirectSearchOptimizer.java

Author: luc
Date: Mon Sep 10 01:22:54 2007
New Revision: 574168

URL: http://svn.apache.org/viewvc?rev=574168&view=rev
Log:
removed an unneeded else

Modified:
    commons/proper/math/trunk/src/java/org/apache/commons/math/optimization/DirectSearchOptimizer.java

Modified: commons/proper/math/trunk/src/java/org/apache/commons/math/optimization/DirectSearchOptimizer.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/java/org/apache/commons/math/optimization/DirectSearchOptimizer.java?rev=574168&r1=574167&r2=574168&view=diff
==============================================================================
--- commons/proper/math/trunk/src/java/org/apache/commons/math/optimization/DirectSearchOptimizer.java (original)
+++ commons/proper/math/trunk/src/java/org/apache/commons/math/optimization/DirectSearchOptimizer.java Mon Sep 10 01:22:54 2007
@@ -573,11 +573,10 @@
                 return (o2 == null) ? 0 : +1;
             } else if (o2 == null) {
                 return -1;
-            } else {
-                double cost1 = ((PointCostPair) o1).cost;
-                double cost2 = ((PointCostPair) o2).cost;
-                return (cost1 < cost2) ? -1 : ((o1 == o2) ? 0 : +1);
             }
+            double cost1 = ((PointCostPair) o1).cost;
+            double cost2 = ((PointCostPair) o2).cost;
+            return (cost1 < cost2) ? -1 : ((o1 == o2) ? 0 : +1);
         }
     };