commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From er...@apache.org
Subject svn commit: r1139460 - /commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BaseSecantSolver.java
Date Fri, 24 Jun 2011 23:13:01 GMT
Author: erans
Date: Fri Jun 24 23:13:00 2011
New Revision: 1139460

URL: http://svn.apache.org/viewvc?rev=1139460&view=rev
Log:
MATH-599
Added brackets around single-statement conditional branches.

Modified:
    commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BaseSecantSolver.java

Modified: commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BaseSecantSolver.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BaseSecantSolver.java?rev=1139460&r1=1139459&r2=1139460&view=diff
==============================================================================
--- commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BaseSecantSolver.java
(original)
+++ commons/proper/math/trunk/src/main/java/org/apache/commons/math/analysis/solvers/BaseSecantSolver.java
Fri Jun 24 23:13:00 2011
@@ -86,8 +86,12 @@ public abstract class BaseSecantSolver e
         // If one of the bounds is the exact root, return it. Since these are
         // not under-approximations or over-approximations, we can return them
         // regardless of the allowed solutions.
-        if (f0 == 0.0) return x0;
-        if (f1 == 0.0) return x1;
+        if (f0 == 0.0) {
+            return x0;
+        }
+        if (f1 == 0.0) {
+            return x1;
+        }
 
         // Verify bracketing of initial solution.
         verifyBracketing(x0, x1);
@@ -115,7 +119,9 @@ public abstract class BaseSecantSolver e
             // If the new approximation is the exact root, return it. Since
             // this is not an under-approximation or an over-approximation,
             // we can return it regardless of the allowed solutions.
-            if (fx == 0.0) return x;
+            if (fx == 0.0) {
+                return x;
+            }
 
             // Update the bounds with the new approximation.
             if (method == Method.SECANT) {
@@ -134,8 +140,12 @@ public abstract class BaseSecantSolver e
                 inverted = !inverted;
             } else {
                 // We had [x0..x1]. We update it to [x0, x].
-                if (method == Method.ILLINOIS) f0 *= 0.5;
-                if (method == Method.PEGASUS) f0 *= f1 / (f1 + fx);
+                if (method == Method.ILLINOIS) {
+                    f0 *= 0.5;
+                }
+                if (method == Method.PEGASUS) {
+                    f0 *= f1 / (f1 + fx);
+                }
                 x1 = x;
                 f1 = fx;
             }



Mime
View raw message