Re: svn commit: r1139126 - in /commons/proper/math/trunk/src: main/java/org/apache/commons/math/analysis/solvers/ site/xdoc/ test/java/org/apache/commons/math/analysis/ test/java/org/apache/commons/math/analysis/solvers/

[Luc Maisonobe]

Hi all,

The provided patch for MATH-599 includes a number of no-brace if 
statements like the following ones:

+                if (method == Method.ILLINOIS) f0 *= 0.5;
+                if (method == Method.PEGASUS) f0 *= f1 / (f1 + fx);

I'm not sure if we have an unwritten rule for this, but personally I 
dislike this style a lot. Checkstyle provides a NeedBraces rule to avoid 
this.

What about activating this rule ?

Luc

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org