<div dir="ltr">Apparently, the check is not completely aligned with the preferences of the community. The relevant rule in the <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_docs_CodingStandards.html-23namespace-2Dindentation&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=cqdM0zW-U9TTlcR6ZxlXqFQYsUyo0N_0Uc_oan6Au74&s=8pYPDID0Qba3IE4EGphOVAC11M-MjajaDYMCj07aOjU&e=">LLVM Coding Standards</a> doesn't mandate the use of the namespace ending comments and it uses a slightly different wording for them ("end namespace xxx" or "end anonymous namespace"). There's also no recommended minimum namespace size, after which the comments should be used. Thus the check first needs to be updated according with the coding standards and community preferences (and maybe the relevant rule of the coding standards should be made a bit more specific, so that it's possible to implement in the check).<div><br><div><div><span style="font-size:12.8px">I already submitted a cleanup patch for this check once (r240270), but had to revert it later due to the reasons above.</span><br></div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 28, 2015 at 10:17 PM, Eugene Zelenko <span dir="ltr"><<a href="mailto:eugene.zelenko@gmail.com" target="_blank">eugene.zelenko@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Eugene.Zelenko added a comment.<br>
<br>
I think it's better to place comments consistently independently from number of declarations in namespace.<br>
<br>
At least Clang-tidy llvm-namespace-comment will be happy and will not complain on same part of code in future.<br>
<div><div><br>
<br>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D11548&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=mQ4LZ2PUj9hpadE3cDHZnIdEwhEBrbAstXeMaFoB9tg&m=cqdM0zW-U9TTlcR6ZxlXqFQYsUyo0N_0Uc_oan6Au74&s=ErFKY2zOq-KA2ccOVYsgtZKtLp-pBLSVNsudAs1caxI&e=" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11548</a><br>
<br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>
</div></div></div></div></div>