<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On 2009-12-01, at 21:48, Mark Rowe wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On 2009-12-01, at 21:43, Peter Kasting wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">I'd like formal clarification about how we enforce our style guide.<div><br></div><div>On&nbsp;<a href="https://bugs.webkit.org/show_bug.cgi?id=31806#c5">https://bugs.webkit.org/show_bug.cgi?id=31806#c5</a> , Yong Li was notified that one of his patches violates a WebKit style rule (no {} on one-line conditional bodies). &nbsp;He objected to the rule, and George Staikos commented that the style guide rules were "guidelines and not hard rules".</div>
<div><br></div><div>I'm not interested in having debates on individual bugs when people don't like particular style guide rules. &nbsp;I don't happen to like this rule either, but my impression is that we simply do not accept willful style guide violations. &nbsp;Can someone with more standing in the project than me confirm or deny that?</div></blockquote><br></div><div>If individual contributors could pick and choose which rules they applied based on their personal preference there would be little point in <i>having</i>&nbsp;the project-wide guidelines. &nbsp;They exist so that personal preference doesn't need to be considered from patch to patch, only when adopting new style guidelines for areas not previously covered.</div></div></blockquote><br></div><div>And to quote from &lt;<a href="http://webkit.org/coding/contributing.html">http://webkit.org/coding/contributing.html</a>&gt;:</div><div><br></div><div><div><blockquote type="cite">In order for your patch to be landed, it's necessary that it comply to the&nbsp;<a href="http://webkit.org/coding/coding-style.html">code style guidelines</a>.</blockquote><br></div><div>There isn't much room for interpretation in that :-)</div><div><br></div><div>- Mark</div><div><br></div></div></body></html>