Indentation for wrapped line

[jltha]

Under what circumstances do we have 8 spaces for our wrapped lines, and 4 spaces for other situations?

My team encountered a problem where checkstyle gives an error, saying that indentation should be 12 lines, e.g.

    public void foo() {
        String[] strings = {"a very long statement", 
            "another very long statement",};
    }

is accepted by checkstyle

vs

    public void foo() {
        String[] strings = {"a very long statement", 
                "another very long statement",};
    }

The module website mentions 8 spaces, which was what I originally thought to be correct, until checkstyle gave errors for some code which has wrapped line.

[okkhoy]

Hmmm.. let me check the checkstyle and update.

[warrencxw]

Since it has been 5 days since the issue was last brought up, can I check if there any updates to this @okkhoy? My team is also facing this issue and we would like to know what we should do in this situation, whether to keep to the checkstyle's 4 spacing for now, or to use 8 spaces and modify checkstyle instead.

[okkhoy]

Thanks for reminding; Hmmm... this is an issue due to my miss. I didn't incorporate the fix before publishing the tP repo 😕

In this case:

• You can either use 4-space indentation for subsequent levels (easier option). [Gradle checks may fail, but marks won't be deducted for code quality in this case.]
• Alternatively, you can tweak the checkstyle file to allow 8-space indentation