<div dir="ltr">Pushed now, thanks everyone for your time!</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Dec 7, 2021 at 2:30 PM James Y Knight <<a href="mailto:jyknight@google.com">jyknight@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">I accepted the revision. Please submit, so we can put this issue to rest. :)<div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Dec 7, 2021 at 6:52 AM Carlos Galvez via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">> especially if clang-tidy does the same job.<div>Thanks, I forgot clang-tidy also enforces this. I've played with it and it will <a href="https://godbolt.org/z/z6d5KnhaG" target="_blank">change the existing style</a> if you have a typo on the namespace.</div><div><br></div><div>> This is one of those things that aren't worth having a patch just for this.</div><div>Fully agree, I meant as part of the pre-commit "git clang-format" that we anyway do on regular patches. But I can see how even that can cause significant noise during review.</div><div><br></div><div>So to summarize the comments so far:</div><div><ul><li>Most predominant style is "// namespace".</li><li>Style in Coding Guidelines should be consistent with clang-format and clang-tidy.</li><li>Regardless of what it says in the Coding Guidelines, both styles are OK and achieve the same goal -> reviewers should not request changes here.</li><li>It's not worth changing existing code.<br></li></ul><div>Based on that, would you be OK with proceeding with the <a href="https://reviews.llvm.org/D115115" target="_blank">patch</a> to update the examples in the Coding Guidelines? If not, do you have any suggestions on what should be improved, or should I just drop it altogether?</div><div><br></div><div>Best regards,</div><div>Carlos</div><div><br></div></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Dec 7, 2021 at 11:50 AM Renato Golin <<a href="mailto:rengolin@gmail.com" target="_blank">rengolin@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">On Tue, 7 Dec 2021 at 10:07, Carlos Galvez via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>I personally thing this is just something clang-format should update automatically as files get modified - over time the style automatically becomes consistent.</div></div></blockquote><div><br></div><div>No, clang-format shouldn't be changing the actual contents as much as it can, especially if clang-tidy does the same job.</div><div><br></div><div>IIUC, James proposal was to match the documentation with whatever clang-tidy emits, whichever way it goes. </div><div><br></div><div>The meaning of all versions are exactly the same and it doesn't really matter which way you have it.</div><div><br></div><div>I echo some comments here that we should focus on bigger things.</div><div><br></div><div>The only action I would take here is to update either the docs, or clang-tidy, to match each other and the general current preference, which seems to be "// namespace" (the blue bars).</div><div><br></div><div>I would strongly discourage people patching existing sources to match the style just for the sake of style. This is one of those things that aren't worth having a patch just for this.</div><div><br></div><div>cheers,</div><div>--renato</div></div></div>
</blockquote></div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>
</blockquote></div>