<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 29, 2016 at 3:54 PM, Eugene Zelenko via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi, David!<br>
<span class=""><br>
On Fri, Apr 29, 2016 at 3:49 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
><br>
><br>
> On Fri, Apr 29, 2016 at 12:47 PM, Eugene Zelenko via llvm-commits<br>
> <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
>><br>
</span><span class="">>> -}<br>
>> +} // end namespace lld<br>
><br>
> Do we have any significantly prevailing style for end of namespace comments?<br>
> (does the style guide say we should have them? Do we usually use "end blah"<br>
> rather than just "blah", etc) - I'd be a bit surprised. & if we don't, I'd<br>
> hesitate to do any cleanup like this.<br>
<br>
</span>There are Clang-tidy check for LLVM end of namespace comments<br>
(<a href="http://clang.llvm.org/extra/clang-tidy/checks/llvm-namespace-comment.html" rel="noreferrer" target="_blank">http://clang.llvm.org/extra/clang-tidy/checks/llvm-namespace-comment.html</a>)<br>
<br>
But as far as I know there is no agreement on what it should be. I<br>
encountered a lot comments with "end" in LLVM headers, so I use this<br>
particular style.<br></blockquote><div><br></div><div>Sure - that's a fine motivation for any code you write in LLVM - but not sure it's necessary/appropriate to change existing comments in code you aren't working on otherwise.<br><br>A check that could correct /incorrect/ ending comments might be handy (if someone uses the wrong namespace name, for example), but unless we've got a pretty clear goal/style guide/buy in I think it's best not to go around doing mass cleanups as yet.<br><br>- David</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
Eugene.<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>