On 6/24/11 2:13 AM, Gilles Sadowski wrote:
> Hi.
>
>>> I agree with you that braces should always be used. Personally, I do
>>> have one exception though, and that is if the statement following it is
>>> on the same line. That way, it is one line, instead of 3 lines, which
>>> makes it more readable. If the 'if' statement spans multiple lines, I
>>> always include braces, even if there is only one single statement
>>> involved. But that is just me...
>> So we have at least two different opinions here, which is good.
>> Let's see what other people think.
> I prefer to have _one_ rule for code formatting.
> [Also, you nvere know when you might need an additional statement wihtin the
> conditional branch (e.g. when debugging).]

+1

Phil
>>> I think the rule is already active in Checkstyle. I think I just chose
>>> to ignore it here.
>> No, the rule is not active in our checkstyle configuration. I have
>> tested it, there are about 130 violations of this rule in our code
>> base, so it is really simple to fix all of them.
> +1 for enabling the rule.
>
>> [...]
> Best regards,
> Gilles
>
> ---------------------------------------------------------------------
> 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

Reply via email to