[ checkstyle-Bugs-2010652 ] arrays and incorrect indentation

View: New views
1 Messages — Rating Filter:   Alert me  

[ checkstyle-Bugs-2010652 ] arrays and incorrect indentation

by SourceForge.net :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Bugs item #2010652, was opened at 2008-07-04 22:31
Message generated for change (Comment added) made by xmlman
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=397078&aid=2010652&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: None
Group: release 4.4
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: feniks (feniks007)
Assigned to: Nobody/Anonymous (nobody)
Summary: arrays and incorrect indentation

Initial Comment:
Hello,

I think there is a problem with the Indentation test when using arrays.
The following line gives a problem (see attachment for complete code example):

test.getArrayOfStrings()[0] = "A string";

I'm using the Eclipse checkstyle plugin, but that imo just a gui for checkstyle, and the problem is in checkstyle itself.

Under Miscellaneous -> Indentation, I use the default check (basicOffset=4, braceAdjustment=0, caseInd=4).

When the line is indented on 8 positions (what we use) I get this message:
Indentation: method call child at indentation level 8 not at correct indentation, 12
If I indent it at 12 position, I get another message:
Indentation: method def child at indentation level 12 not at correct indentation, 8

There does not seem to be any way to validate the line, so I think this is a bug.



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

Comment By: Kyung Tae Kang (xmlman)
Date: 2009-10-29 19:23

Message:
Hi.
I resolved the above method call's indentation bug.
This bug happens because there is a collision between Method Call's
Indentation and Assingment Indentation.
Our companay uses Checkstyle 4.4. So if you use Checkstyle 4.4 I can send
checkstyle-all-4.4.jar or checkstyle-4.4.jar.

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

Comment By: feniks (feniks007)
Date: 2008-11-07 23:54

Message:
File Added: IndentProblem.java

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

Comment By: feniks (feniks007)
Date: 2008-11-07 23:52

Message:
In fact it's not only when using arrays the same type of problem can occur
with simple classes.
I'll attach another file that proves this.


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

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

------------------------------------------------------------------------------
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference
_______________________________________________
Checkstyle-devel mailing list
Checkstyle-devel@...
https://lists.sourceforge.net/lists/listinfo/checkstyle-devel