<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Nov 20, 2010, at 10:01 AM, Chris Lattner 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 Nov 18, 2010, at 11:53 PM, Chandler Carruth wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote">On Thu, Nov 18, 2010 at 8:18 AM, Howard Hinnant <span dir="ltr"><<a href="mailto:hhinnant@apple.com">hhinnant@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div id=":cu2">Ok, if that is the consensus of the community I can easily remove them. On a previous product this was a customer-requested feature and subsequent feedback on it was positive.<br>
<br>
I encourage people to view this question through actual use of the header, as opposed to a svn diff.</div></blockquote></div><br><div>I'm actually pretty positive on them, but I wonder if using the (somewhat more annoying to maintain) '//' prefix comments would help avoid confusion? That would provide an ever-present reminder, even in diffs, that this is merely a comment. On the other side, it makes formatting etc more annoying so I understand that it may not be the right tradeoff.</div></blockquote><br></div><div>I agree that it would make it easier to read/know. On the other hand, /* */ comments have a nice vectorized comment skipper and // don't (and can't reasonably). It probably isn't measurable, but this means that // comments are slower to lex than /* comments :-)</div></div></blockquote><br></div><div>If you're really concerned about this, you *can* put BCPL comments inside C comments. I, too, frequently find myself confused when looking at these headers because of the lengthy comments-than-don't-look-like-comments.</div><div><br></div><div>John.</div><br></body></html>