<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&#39;t think either of these are unwritten rules.  Both (especially the second) are frequently seen with two spaces.  I frequently see &quot;close of #if or namespace&quot; EOL comments with one space before //, and other EOL comments with two.  I wouldn&#39;t mind standardizing (my preference is two spaces) but I&#39;m not sure we can yet.</div>
<div><br></div><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="&#39;courier new&#39;, monospace">if (condition1</font><div>
<font face="&#39;courier new&#39;, monospace">        &amp;&amp; condition2)</font></div><div><font face="&#39;courier new&#39;, monospace">    statement;</font></div></blockquote></div></div></blockquote><div><br></div>
<div>I note you used &quot;may&quot; instead of &quot;should&quot; or &quot;must&quot;.  It seems like we should pick the right thing and say &quot;must&quot;.  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="&#39;courier new&#39;, monospace">if (long_condition1</font></div><div><font class="Apple-style-span" face="&#39;courier new&#39;, monospace">    &amp;&amp; long_condition2</font></div>
<div><font class="Apple-style-span" face="&#39;courier new&#39;, monospace">    &amp;&amp; long_condition3)</font></div><div><font class="Apple-style-span" face="&#39;courier new&#39;, monospace">    statement;</font></div>
<div><font class="Apple-style-span" face="&#39;courier new&#39;, monospace"><br></font></div><div><font class="Apple-style-span" face="arial, helvetica, sans-serif">I don&#39;t believe I&#39;ve seen cases like the one you give.</font></div>
<div><font class="Apple-style-span" face="&#39;courier new&#39;, monospace"><br></font></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div><div><blockquote style="margin:0 0 0 40px;border:none;padding:0px">
<div><font face="&#39;courier new&#39;, monospace"></font></div></blockquote></div><div><b>Misc</b></div><div><font face="&#39;courier new&#39;, monospace"><span style="font-family:arial"><div>Files who should end with newlines.</div>
</span></font></div></div></blockquote><div><br></div><div>s/who//.  In fact it might be clearer to say &quot;Files should end with a trailing newline&quot;.</div><div><br></div><div>PK </div></div>