logo       

[ checkstyle-Bugs-1841732 ] Indentation::caseIndent not useful: msg#00285

Subject: [ checkstyle-Bugs-1841732 ] Indentation::caseIndent not useful
Bugs item #1841732, was opened at 2007-11-30 09:21
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397078&aid=1841732&group_id=29721

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: usage
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Austin Hastings (austin_hastings)
Assigned to: Nobody/Anonymous (nobody)
Summary: Indentation::caseIndent not useful

Initial Comment:
My style objective is to align case labels with their switch statement, which 
is aligned with the enclosing block (of the switch):

.....switch (foo)
.....{
.....case 1: return mumble;
.....case 2: return bumble;
.....default: whatever();
.....}

I have code aligned like this, but if I set caseIndent = 0 then I get 

case child at indentation level 12 not at correct indentation, 16

while if I set caseIndent = -4 then I get

case child at indentation level 12 not at correct indentation, 8

It doesn't seem to accept a negative-zero option, so there's some kind of 
schizophrenia involved - negatives are subtracted from 12 but positives are 
added to 16? 

Anyway, she's a no work!

Thanks,

=Austin


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397078&aid=1841732&group_id=29721

-------------------------------------------------------------------------
SF.Net email is sponsored by: The Future of Linux Business White Paper
from Novell.  From the desktop to the data center, Linux is going
mainstream.  Let it simplify your IT future.
http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4


<Prev in Thread] Current Thread [Next in Thread>