malcolm.parsons added inline comments.
================ Comment at: test/clang-tidy/modernize-use-default.cpp:142 +// CHECK-MESSAGES: :[[@LINE-2]]:1: warning: use '= default' +// CHECK-FIXES: TempODef<T>::~TempODef() = default; + ---------------- aaron.ballman wrote: > Would it be better if the `= default` were on the inline declaration rather > than the out-of-line definition? If so, that might be a reasonable FIXME to > add to this test (assuming you don't feel like fixing it as part of this > patch). > > Also, does the `= delete` check suffer from the same issue? There are valid reasons for having out-of-line definitions, even when defaulted. The delete check only triggers if there is no definition. Template instantiation can't copy a definition that doesn't exist. https://reviews.llvm.org/D26582 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
