Bug 27508 – Add checks for switch statement indentation to cpplint

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Jakob Petsovits]

Created attachment 33196 [details]
Add checks for switch statement indentation to cpplint

Switch statements have strong similarities to namespaces when it comes to indentation, so I stole from my previous patch (https://bugs.webkit.org/show_bug.cgi?id=27461) and adapted some parts to implement style guideline #5 from http://webkit.org/coding/coding-style.html to cpplint. Looking for a motivated reviewer, hope you like it!

[Jakob Petsovits]

Created attachment 33197 [details]
Add checks for switch statement indentation to cpplint

Note to self: Don't post patch files unless bug report is created and URL is inserted into the ChangeLog.
Apart from that, same patch as the previous one.

[David Levin]

Re changelog: Just use prepare-ChangeLog --bug YourBugNumber

[David Levin]

Comment on attachment 33197 [details]
Add checks for switch statement indentation to cpplint

In general it looks good but I think it will have problems on nested switch statements. What can be done about that?

[Jakob Petsovits]

Created attachment 33200 [details]
Add checks for switch statement indentation to cpplint (try 3)

Good point, I had not thought of that case. Here's a relatively simple solution that'll work in pretty much all common cases, approach explained in the code comments. Plus more test cases. Is that good enough, or does it need to be perfect?

[David Levin]

Comment on attachment 33200 [details]
Add checks for switch statement indentation to cpplint (try 3)

This is an improvement over what we have now and seems to minimize the possibility of false alarms (with respect to nested switch statements), so it seems like a positive step forward.

[David Levin]

Committed as http://trac.webkit.org/changeset/46189