#587 Ignore wrapped lines for indentation check

open
nobody
Check (274)
5
2013-11-26
2011-10-21
Julien HENRY
No

Hi,

In my company we have many violations of the indentation check on wrapped lines. I have seen lot of feature request, and discussions on the ML. My understanding is that is is a very complicated issue, because there are many different usage that would require to add a lot of parameters to the indentation check.
Waiting for an eventual improvement, here is my proposal: add a single parameter "Ignore wrapped lines". This way people can wrap/indent lines the way they want (or use Eclipse reformat feature) but still we could activate IndentationCheck.
I feel this could be easier to implement than supporting all use case of line wrapping.

Before stating to work on a patch I would like to know if this is something you would consider to merge?

Regards

Discussion

  • Scott R Duchin
    Scott R Duchin
    2013-11-26

    I cannot use the Indentation Check as currently implemented. I suspect many cannot use it as well as it does not support the official java style guide.
    to fix, add another parameter for "wrappedOffset" or "continuationOffset"
    all the editors allow you to set this offset.
    in my company we indent by 2 and continuation by 4
    official java style guide indents by 4 and continuation by 8