On Dec 1, 2009, at 9:43 PM, Peter Kasting wrote:

I'd like formal clarification about how we enforce our style guide.

On https://bugs.webkit.org/show_bug.cgi?id=31806#c5 , Yong Li was notified that one of his patches violates a WebKit style rule (no {} on one-line conditional bodies). He objected to the rule, and George Staikos commented that the style guide rules were "guidelines and not hard rules".

I'm not interested in having debates on individual bugs when people don't like particular style guide rules. I don't happen to like this rule either, but my impression is that we simply do not accept willful style guide violations. Can someone with more standing in the project than me confirm or deny that?

Some rules in the style guide may call for judgment and occasionally admit exceptions, like the naming rule. But this one seems pretty clear-cut.

Note: if people don't like this rule we can discuss changing the style guide. But in generally people should not be picking and choosing which style rules to follow individually. The point is to make the code base look consistent.

Regards,
Maciej

_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev

Reply via email to