[VOTE] indent on case
In our code guidelines (or at least the IDE setup files), case is not indented switch (i) { case 1: break; but the checkstyle profile is looking for a 4 character indent: switch (i) { case 1: break; which do we want? [ ] No indent [ ] Indent 4 characters -- Jeremy - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [VOTE] indent on case
My vote On Aug 11, 2006, at 5:55 PM, Jeremy Boynes wrote: In our code guidelines (or at least the IDE setup files), case is not indented switch (i) { case 1: break; [X] No indent [ ] Indent 4 characters -- Jeremy - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [VOTE] indent on case
I'm note sure this requires a vote but I prefer indent 4. Jim On Aug 11, 2006, at 5:55 PM, Jeremy Boynes wrote: In our code guidelines (or at least the IDE setup files), case is not indented switch (i) { case 1: break; but the checkstyle profile is looking for a 4 character indent: switch (i) { case 1: break; which do we want? [ ] No indent [ ] Indent 4 characters -- Jeremy - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [VOTE] indent on case
"switch" clause may have statements other than "case", and people normally indent sub elements... -- Yang ZHONG
Re: [VOTE] indent on case
The other elements would go at the level of the "break" - indent 4 or indent 8 respectively. That's actually why I prefer the "no-indent" case as it puts the other elements at the same indent as if you were using an "if" block in rather than an additional 4 characters in. -- Jeremy On Aug 11, 2006, at 6:46 PM, Yang ZHONG wrote: "switch" clause may have statements other than "case", and people normally indent sub elements... -- Yang ZHONG - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [VOTE] indent on case
I like indent on case in line with the rest of the indenting policy. From: Jim Marino <[EMAIL PROTECTED]> Reply-To: tuscany-dev@ws.apache.org To: tuscany-dev@ws.apache.org Subject: Re: [VOTE] indent on case Date: Fri, 11 Aug 2006 18:31:58 -0700 I'm note sure this requires a vote but I prefer indent 4. Jim On Aug 11, 2006, at 5:55 PM, Jeremy Boynes wrote: In our code guidelines (or at least the IDE setup files), case is not indented switch (i) { case 1: break; but the checkstyle profile is looking for a 4 character indent: switch (i) { case 1: break; which do we want? [ ] No indent [ ] Indent 4 characters -- Jeremy - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] _ Be the first to hear what's new at MSN - sign up to our free newsletters! http://www.msn.co.uk/newsletters - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: [VOTE] indent on case
i'm in the indent camp On 12/08/06, Meeraj Kunnumpurath <[EMAIL PROTECTED]> wrote: I like indent on case in line with the rest of the indenting policy. >From: Jim Marino <[EMAIL PROTECTED]> >Reply-To: tuscany-dev@ws.apache.org >To: tuscany-dev@ws.apache.org >Subject: Re: [VOTE] indent on case >Date: Fri, 11 Aug 2006 18:31:58 -0700 > >I'm note sure this requires a vote but I prefer indent 4. > >Jim > > >On Aug 11, 2006, at 5:55 PM, Jeremy Boynes wrote: > >>In our code guidelines (or at least the IDE setup files), case is not >>indented >> switch (i) { >> case 1: >> break; >> >>but the checkstyle profile is looking for a 4 character indent: >> switch (i) { >> case 1: >> break; >> >>which do we want? >> >>[ ] No indent >>[ ] Indent 4 characters >> >>-- >>Jeremy >> >>- >>To unsubscribe, e-mail: [EMAIL PROTECTED] >>For additional commands, e-mail: [EMAIL PROTECTED] >> > > >- >To unsubscribe, e-mail: [EMAIL PROTECTED] >For additional commands, e-mail: [EMAIL PROTECTED] > _ Be the first to hear what's new at MSN - sign up to our free newsletters! http://www.msn.co.uk/newsletters - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED] -- Best Regards Kelvin Goodson