<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-02, at 21:46, Peter Kasting wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote">On Wed, Dec 2, 2009 at 9:19 PM, Mark Rowe <span dir="ltr">&lt;<a href="mailto:mrowe@apple.com">mrowe@apple.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On 2009-12-02, at 21:00, Peter Kasting wrote:<br>
<br>
&gt; I find this tricky to read and error-prone. &nbsp;I propose that the rule be modified to be:<br>
&gt;<br>
&gt; * When all arms of a conditional or loop are one physical line, do not use braces. &nbsp;If any arms are more than one physical line (even if they are one logical line), use braces on all arms.<br>
<br>
</div>I do not agree that this would be an improvement.<br></blockquote><div><br></div><div>Are you satisfied with the existing rule, then?</div></div></blockquote><div><br></div><div>I am satisfied with the existing rule, and I have seen nothing to support the suggestion that a change to the rule would provide additional benefits.</div><div><br></div><blockquote type="cite"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; "><div class="im">&gt; In most places this will not differ from the existing code, so it will not "cause the whole codebase to become invalid";<br>
<br>
</div>I glanced briefly at three files: RenderObject.cpp, Element.cpp and Node.cpp. &nbsp;In these three files I counted over two dozen places that would require modification to conform with this new rule. &nbsp;That's by no means a majority of the relevant statements in these files, but it's not a small number either.</blockquote>
<div><br></div><div>I stand by my statement.&nbsp;</div></div></blockquote><br></div><div>I was not putting this forward as my reasoning for not changing the style, merely as a data point.</div><div><br></div><div>- Mark</div><div><br></div></body></html>