<p dir="ltr">Hmm, right now there is no way to use alpha checks in clang tidy? <br>
Piotr</p>
<div class="gmail_extra"><br><div class="gmail_quote">On Nov 10, 2016 10:57, "Malcolm Parsons via cfe-commits" <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 9 November 2016 at 18:50, Devin Coughlin <<a href="mailto:dcoughlin@apple.com">dcoughlin@apple.com</a>> wrote:<br>
<br>
> We agree that this is a valuable checker and are committed to getting it out of alpha. This check is in alpha because:<br>
><br>
> a) The diagnostic experience is not very good. It reports a call path directly in the diagnostic message (for example “Call path: foo <— bar” for a call to foo() in bar()) rather than as a path diagnostic.<br>
<br>
Agreed.<br>
<br>
> b) The lack of path-sensitive reasoning may result in false positives when a called function uses a member variable flag to track whether initialization is complete and does not call the virtual member function during initialization.<br>
<br>
Right, we're not doing this.<br>
<br>
> c) The check issues a warning for both calls to pure virtual functions (which is always an error) and non-pure virtual functions (which is more of a code smell and may be a false positive).<br>
<br>
I'm using static analysis to find code smells.<br>
<br>
> I’ll commit to doing Step 1) in the immediate future and Step 2) eventually. Once the checker is on by default we’ll need to assess whether the false positive rate from c) is too high — if so, we’ll need to turn the non-pure-virtual case off by default.<br>
<br>
LGTM.<br>
<br>
Thanks,<br>
--<br>
Malcolm Parsons<br>
______________________________<wbr>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-commits</a><br>
</blockquote></div></div>