
<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Hi Reid,<div><br></div><div>Thanks for reviewing.</div><div><br><div><div>On Oct 30, 2013, at 1:12 PM, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr">While you're moving this code around, it should probably follow the convention for namespaces:<div><a href="http://llvm.org/docs/CodingStandards.html#namespace-indentation">http://llvm.org/docs/CodingStandards.html#namespace-indentation</a><br></div></div></blockquote><div><br></div>Yes, that’s a good point. Fixed.</div><div><br><blockquote type="cite"><div dir="ltr"><div></div><div><br></div><div>Otherwise, go for it.</div></div></blockquote><div><br></div>Committed in r193717.</div></div><div><br></div><div>Mark</div><div><br></div></body></html>