<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div><div>On Sep 3, 2009, at 3:51 PM, Peter Kasting wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote">On Thu, Sep 3, 2009 at 3:46 PM, Darin Adler <span dir="ltr">&lt;<a href="mailto:darin@apple.com">darin@apple.com</a>&gt;</span> wrote:<br><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 style="word-wrap:break-word"><div><div>On Sep 3, 2009, at 8:43 AM, David Levin wrote:</div><div class="im"><div><br></div><blockquote type="cite"><div><div class="gmail_quote"><div><div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"> <div>There should only be a single space before end of line comments. <span style="background-color:rgb(102, 255, 153)">An exception is if one is lining up several end of line comments.</span></div></blockquote></div></div> </div></div></blockquote><div><br></div></div>I donít think we should have this exception.</div><div><br></div><div>Things manually lined up in source code generally create a small maintainability problem. You canít rename things without re-lining things up, and if you make other types code changes without noticing the lined-up comments the code ends up looking messy an peculiar.</div> </div></blockquote><div><br></div><div>The common place to do this is on a set of constants/enum values, where I don't think the objections you raise are a big deal.</div></div></blockquote><div><br></div><div>I don't think we normally allow an exception for this.</div><div><br></div><div>(Personally I'd quite like it if we did, I prefer to constants aligned as you suggest Ė but rules is rules &amp; this has not been allowed for me in the past).</div><div><br></div>G.</div><div><br><blockquote type="cite"><div class="gmail_quote"><div><br></div><div>PK</div></div> _______________________________________________<br>webkit-dev mailing list<br><a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev<br></blockquote></div><br></div></body></html>