The style script flagged an issue in my code yesterday for an issue I didn't even know existed. How do you indent case clauses for a switch statement? The WebKit style states that case clauses have the same indentation as their switch. I HATE that style. And I had no idea that was the WebKit style. I use the "indent the case" style and have never had anyone flag it in the past. Without getting into style religion, I was looking at the code and it seems that there are many more uses of the "indent the case" style than the "correct" style.
Maybe we could change the style rule in the interest of changing fewer files (and because I think it generally reads better)?
I'm fine with changing my code to match the style. But the style script is going to be kicking out a lot of these errors and I think we should make sure we want to go down this road before that happens.
----- ~Chris cmar...@apple.com _______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
