<div dir="ltr">You probably want to send this to cfe-commits instead of llvm-commits</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 21, 2016 at 6:43 AM, Robert Bolter 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">





<div lang="EN-GB" link="blue" vlink="purple">
<div>
<p class="MsoNormal">Hi,<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">First time poster here, Please advise…<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Can I contribute these patches for clang-tidy modernize-use-override addressing two problems:<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">1: missing spaces on pure function decls<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Orig:<u></u><u></u></p>
<p class="MsoNormal" style="margin-left:36.0pt">void pure() const=0<u></u><u></u></p>
<p class="MsoNormal">Problem:<u></u><u></u></p>
<p class="MsoNormal" style="margin-left:36.0pt">void pure() constoverride =0<u></u><u></u></p>
<p class="MsoNormal">Fixed:<u></u><u></u></p>
<p class="MsoNormal" style="text-indent:36.0pt">void pure() const override =0<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">2: This is ms-extension specific, but possibly applies to other attribute types as I see incorrect placement of override for Inherited Attributes and atts located before the method
<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Orig:<u></u><u></u></p>
<p class="MsoNormal">class __declspec(dllexport) X : public Y<u></u><u></u></p>
<p class="MsoNormal">  {<u></u><u></u></p>
<p class="MsoNormal">void p(); //inherits decl att<u></u><u></u></p>
<p class="MsoNormal">  };<u></u><u></u></p>
<p class="MsoNormal">Problem:<u></u><u></u></p>
<p class="MsoNormal">class override __declspec(dllexport) class X : public Y<u></u><u></u></p>
<p class="MsoNormal">  {<u></u><u></u></p>
<p class="MsoNormal">  void p();<u></u><u></u></p>
<p class="MsoNormal">  };<u></u><u></u></p>
<p class="MsoNormal">Fixed:<u></u><u></u></p>
<p class="MsoNormal">class __declspec(dllexport) class X : public Y<u></u><u></u></p>
<p class="MsoNormal">  {<u></u><u></u></p>
<p class="MsoNormal">  void p() override;<u></u><u></u></p>
<p class="MsoNormal">  };<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">I added test/clang-tidy/modernize-use-override-ms.cpp and modified modernize-use-override.cpp to test these fixes.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">I’ve also added a –quiet option to clang-tidy/tool/run-clang-tidy.py  and a progress countdown which I personally find useful
<span style="font-family:Wingdings">J</span><u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Thanks,<u></u><u></u></p>
<p class="MsoNormal">Rob.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<br>
<hr>
<font face="Arial" color="Gray" size="1"><br>
Autodesk Limited<br>
Registered Office: One Discovery Place, Columbus Drive, Farnborough, Hampshire GU14 0NZ<br>
Registered in England and Wales, No. 1839239<br>
</font>
</div>

<br>_______________________________________________<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>
<br></blockquote></div><br></div>