[PATCH] D28287: [clang-tidy] Ignore default arguments in modernize-default-member-init

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 4 09:39:15 PST 2017


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM!



================
Comment at: test/clang-tidy/modernize-use-default-member-init.cpp:224
 
+struct NegativeDefaultArg
+{
----------------
malcolm.parsons wrote:
> aaron.ballman wrote:
> > Can we also have a positive test with an enumerator?
> See lines 159-166.
Ah, thanks, my eyes missed that. :-)


https://reviews.llvm.org/D28287





More information about the cfe-commits mailing list