<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Nov 30, 2011, at 7:54 PM, Anshuman Dasgupta wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">
<meta content="text/html; charset=ISO-8859-1" http-equiv="Content-Type">
<div bgcolor="#FFFFFF" text="#000000">
<tt>Jakob,<br>
<br>
I'm attaching a patch with the changes that you suggested.<br></tt></div></blockquote><div><br></div><div>Almost there. Two things:</div><div><br></div><div><a href="http://llvm.org/docs/CodingStandards.html#micro_namespaceindent">http://llvm.org/docs/CodingStandards.html#micro_namespaceindent</a></div><div><a href="http://llvm.org/docs/CodingStandards.html#micro_anonns">http://llvm.org/docs/CodingStandards.html#micro_anonns</a></div><div><br></div><div>Don't indent for namespaces, and use individual anonymous namespaces for each class.</div><div><br></div><div>Go ahead and commit with those changes.</div><div><br></div><div>/jakob</div><div><br></div></div></body></html>