
<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On May 21, 2011, at 5:31 AM, Matthieu Monrocq wrote:</div><br class="Apple-interchange-newline"><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>It's already implemented, you can check it with cxxRecordDecl->hasAttr<FinalAttr>().</div><div><br></div><div>-Argyrios</div><br><blockquote type="cite"><div class="gmail_quote"><div>

<br>Thanks for looking into this!<br><br>-- Matthieu<br><br> <br></div></div><br>

<span><clang_non_virtual_destructor.diff></span>_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br></blockquote></div><br></body></html>