<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Sep 2, 2009, at 9:46 AM, Peter Kasting wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote">On Wed, Sep 2, 2009 at 8:40 AM, David Levin <span dir="ltr">&lt;<a href="mailto:levin@chromium.org">levin@chromium.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<b><div><span class="Apple-style-span" style="font-weight: normal; "><b><div style="display: inline !important; "><span style="font-weight:normal"><b>Comments</b></span></div></b></span></div>

</b><div>
<div>There should be a <i>single</i> space after punctation and before the next sentence.</div>
<div><br></div><div>There should only be a single space before end of line comments.</div></div></blockquote><div><br></div><div>I don't think either of these are unwritten rules. &nbsp;Both (especially the second) are frequently seen with two spaces. &nbsp;I frequently see "close of #if or namespace" EOL comments with one space before //, and other EOL comments with two. &nbsp;I wouldn't mind standardizing (my preference is two spaces) but I'm not sure we can yet.</div></div></blockquote><div><br></div>We did standardize on one space for both of these years ago. But this rule is one that people often forget and that a few others disagree with and disobey intentionally.<br><blockquote type="cite"><div class="gmail_quote">
</div></blockquote><div class="gmail_quote"><div><br></div></div><blockquote type="cite"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div><div></div><div><b>Indentation</b></div><div>Additional clauses in a conditional may be indented 4 extra spaces to visually separate them from the statement to be executed.</div>








<div><br></div><div><br></div><div>Like this</div><div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><font face="'courier new', monospace">if (condition1</font><div>
<font face="'courier new', monospace">&nbsp;&nbsp; &nbsp; &nbsp; &nbsp;&amp;&amp; condition2)</font></div><div><font face="'courier new', monospace">&nbsp;&nbsp; &nbsp;statement;</font></div></blockquote></div></div></blockquote><div><br></div>
<div>I note you used "may" instead of "should" or "must". &nbsp;It seems like we should pick the right thing and say "must". &nbsp;My reading of the current style guide is that the current recommended indentation for this case is:</div>
<div><font class="Apple-style-span" face="'courier new', monospace">if (long_condition1</font></div><div><font class="Apple-style-span" face="'courier new', monospace">&nbsp;&nbsp; &nbsp;&amp;&amp; long_condition2</font></div>
<div><font class="Apple-style-span" face="'courier new', monospace">&nbsp;&nbsp; &nbsp;&amp;&amp; long_condition3)</font></div><div><font class="Apple-style-span" face="'courier new', monospace">&nbsp;&nbsp; &nbsp;statement;</font></div>
<div><font class="Apple-style-span" face="'courier new', monospace"><br></font></div><div><font class="Apple-style-span" face="arial, helvetica, sans-serif">I don't believe I've seen cases like the one you give.</font></div>
</div></blockquote></div><br><div>The style where the &amp;&amp; and the statement have the same indentation doesn’t work well. David is correct that I recommend and use the extra indenting level, and have included it in new code added to this project for years. We may not have consensus on this, though. Some people still put the operators on the ends of lines, in fact.</div><div><br></div><div>&nbsp;&nbsp; &nbsp;-- Darin</div><div><br></div></body></html>