aaron.ballman accepted this revision. aaron.ballman added a comment. LGTM with a commenting request.
================ Comment at: test/clang-tidy/cppcoreguidelines-pro-type-member-init.cpp:342 @@ +341,3 @@ + +template <typename T> +class PositiveSelfInitialization : NegativeAggregateType ---------------- hokein wrote: > aaron.ballman wrote: > > Is it required to be a templated class to trigger the crash? > Yes, it's required. Then some comments in this test would be good, I would assume otherwise from just looking at the test. http://reviews.llvm.org/D19802 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
