Switch mechanics, indent

[bounceme]

also now that I know about cindent i'm curious if there are other places like this to remove code and increase performance

[amadeus]

Testing this out, I am noticing some weirdness

If I just use the indentation given to me, I get this scenario:

switch (test) {
    case 1:
    test = 2;
    word = 3;
    hi = 4;
    break;
    case 2:
    weird = 2;
    hi = 3;
    break;
    default:
    test = 10;
    word = 20;
    break;
}

Yet if I try to indent the first line, 2 different things will happen:

switch (test) {
    case 1:
        test = 2; // purposely indented this line
    word = 3;
    hi = 20;
    break;
    case 2:
        hi = 2; // Purposely indented this line
        weird = 2;
        strange = 3;
        break;
    default:
        weird = 3; // Purposely indented this line
        hi = 2;
        break;
}

[bounceme]

Your example is actually broken for me too. If I understand right, this is being caused by something to do with the breaks

[bounceme]

This seems to work now and has removed even more code

[davidchambers]

⬅️ ⬅️

[davidchambers]

I ❤️ all the red in the diff!

[bounceme]

also people could customize this in a few ways:

set cino+=:0

[amadeus]

Disregard my entire previously written message, I was testing on the wrong PR.

This seems to work quite well! Of course I'd like to sit on it for a couple days to make sure some other indent didn't get screwed up somehow? I only say that because I am completely ignorant of how the indent stuff works and therefore can't know if it might affect other things or not. Perhaps that's what develop is for though.

[amadeus]

Would you like to hold off on merging this to find some other improvements you can make? Or would prefer to file those as separate PRs?

[bounceme]

i'd prefer fixing whatever bugs come of this as they are found, which with my testing of this, isn't so worrying to prevent a merge.