Sounds good to me.
Geoff
On Dec 2, 2009, at 8:58 PM, Peter Kasting wrote:
> On Wed, Dec 2, 2009 at 8:05 PM, Maciej Stachowiak wrote:
> I believe one rule that could work is something like this:
>
> - Indent case labels inside a switch two spaces.
> - Indent actual statements inside a switch fo
On Wed, Dec 2, 2009 at 8:05 PM, Maciej Stachowiak wrote:
> I believe one rule that could work is something like this:
>
> - Indent case labels inside a switch two spaces.
> - Indent actual statements inside a switch four spaces.
> - In the case where a case label is followed by a block, include t
On Dec 2, 2009, at 7:56 PM, Geoffrey Garen wrote:
The downside is that some code can get indented too far, which is
particularly unfortunate for large switches.
We could issue a fuzzy declaration such as, "Indent case blocks,
except in situations where an unreasonable amount of code would
> The downside is that some code can get indented too far, which is
> particularly unfortunate for large switches.
We could issue a fuzzy declaration such as, "Indent case blocks, except in
situations where an unreasonable amount of code would end up so-indented,
causing readability problems".
On Dec 2, 2009, at 4:47 PM, Alexey Proskuryakov wrote:
On 02.12.2009, at 15:25, Chris Marrin wrote:
Maybe we could change the style rule in the interest of changing
fewer files (and because I think it generally reads better)?
I support changing or dropping this rule. Because of this rule
On 02.12.2009, at 15:25, Chris Marrin wrote:
Maybe we could change the style rule in the interest of changing
fewer files (and because I think it generally reads better)?
I support changing or dropping this rule. Because of this rule, there
is no good way to format cases that need braces,
The style script flagged an issue in my code yesterday for an issue I
didn't even know existed. How do you indent case clauses for a switch
statement? The WebKit style states that case clauses have the same
indentation as their switch. I HATE that style. And I had no idea that
was the Web
7 matches
Mail list logo
