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/

[Sébastien Brisard]

+1

2011/6/24 Luc Maisonobe <luc.maison...@free.fr>:
> 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
>
>

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