<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On May 24, 2011, at 10:43 AM, Matthieu Monrocq wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><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-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; position: static; z-index: auto; ">
<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; position: static; z-index: auto; ">
<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></div></div></blockquote><div><br></div><div>Adding -std=c++0x to the test is fine, even though we handle the 'final' syntax even without passing -std=c++0x.</div><div><br></div><div>Committed your patch in r131989, thanks!</div><div><br></div><div>-Argyrios</div><br><blockquote type="cite"><div class="gmail_quote"><div>
<br>-- Matthieu<br></div></div><br>
<span><clang_non_virtual_destructor.diff></span></blockquote></div><br></body></html>