<br><br><div class="gmail_quote">2011/5/24 Argyrios Kyrtzidis <span dir="ltr"><<a href="mailto:kyrtzidis@apple.com">kyrtzidis@apple.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div style="word-wrap:break-word"><div><div class="im"><div>On May 21, 2011, at 5:31 AM, Matthieu Monrocq wrote:</div><br><blockquote type="cite"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204, 204, 204);border-left-style:solid;padding-left:1ex">
<div style="word-wrap:break-word"><div></div><div>Your new diagnostic for non-virtual destructor looks great and useful, I'd suggest we put it in "most" diagnostic group.</div>
<div><br></div><div>Thanks for your work!</div><div><br></div><font color="#888888"><div>-Argyrios</div></font><br></div></blockquote><div><br>Here is the new patch with the warning wired up in "most".<br><br>I hope that the final attribute for classes will be implemented soon since it'll remove some false positives for this warning.<br>
</div></div></blockquote><div><br></div></div><div>It's already implemented, you can check it with cxxRecordDecl->hasAttr<FinalAttr>().</div><div><br></div><div>-Argyrios</div><br></div></div></blockquote><div>
<br> Ah great! I had completely missed it...<br><br>I have reviewed the patch then, and taken the liberty of pasting the paragraph of the Standard that specifies this just above the little piece of code. I have also added the tests to `destructor.cpp`.<br>
<br>I have ordered the test so that  hasAttr<FinalAttr>  is the last condition, in order to economize the calls (I suppose it's less costly than looking up the destructor).<br><br>Since this means that I now specifies -std=c++0x on the command line (for final), I would understand if it should only be activated conditionally or if the test should be split.<br>
<br>-- Matthieu<br></div></div><br>