<p dir="ltr"><br>
On Apr 10, 2015 6:41 PM, "Alexander Kornienko" <<a href="mailto:alexfh@google.com">alexfh@google.com</a>> wrote:<br>
><br>
> ================<br>
> Comment at: include/clang/Analysis/AnalysisContext.h:327<br>
> @@ -326,3 +326,3 @@<br>
>  public:<br>
> -  ~ScopeContext() {}<br>
> +  ~ScopeContext() override {}<br>
><br>
> ----------------<br>
> dblaikie wrote:<br>
> > Do we have a clang-tidy warning for explicitly writing default definitions of functions? Would love to just not have this sort of code written at all.<br>
> Would you prefer to remove all empty or default destructors defined inline in derived classes? No, we don't have this check yet.</p>
<p dir="ltr">Only if you feel like doing that (consider this precommit review approval if you wanted it) - this override change is fine even if that dtor work isn't done.</p>
<p dir="ltr">><br>
> <a href="http://reviews.llvm.org/D8926">http://reviews.llvm.org/D8926</a><br>
><br>
> EMAIL PREFERENCES<br>
>   <a href="http://reviews.llvm.org/settings/panel/emailpreferences/">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
><br>
><br>
</p>