================
Comment at: include/clang/Analysis/AnalysisContext.h:327
@@ -326,3 +326,3 @@
public:
- ~ScopeContext() {}
+ ~ScopeContext() override {}
----------------
dblaikie wrote:
> 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.
Would you prefer to remove all empty or default destructors defined inline in
derived classes? No, we don't have this check yet.
http://reviews.llvm.org/D8926
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
