<div dir="ltr">LGTM.<div><br></div><div>--renato</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 12 January 2014 11:00, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Currently there is a mixture of indented namespaces and un-indented namespaces in both LLVM and Clang. I think this is confusing and it wastes developer time debating the issue. I'd like to pick one and stick with it consistently.<div>

<br></div><div>Indenting cannot possibly work in many contexts -- file-wide namespaces just make no sense to indent. So I don't think we should pick "always indent".</div><div><br></div><div>The common pattern to indent is when there is a small body of code which is within a namespace. The reason for not indenting large bodies of code is that when the namespace spans so much, the indent loses any structural value, and turns into just a cost of losing valuable horizontal space in which to write code. This makes sense to me.</div>

<div><br></div><div>But for small blocks of code, the indent has relatively low value -- both the start and end are typically visible on the screen. Why bother indenting this special case? The value seems very, very low.</div>

<div><br></div><div>So, I suggest no indent of namespaces ever. I'm happy to make the corresponding change to the coding standards. Naturally, I'm not suggesting rampant re-indenting of code. I'd just like to set a consistent rule going forward so we don't debate this, and know what to do when I see a chunk of code and about to make very significant changes to it and want to clean up formatting while I'm there.</div>
<span class="HOEnZb"><font color="#888888">
<div><br></div><div>-Chandler</div></font></span></div>
<br>_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
<br></blockquote></div><br></div>