[PATCH] D13871: Add modernize-use-default check to clang-tidy.

Manuel Klimek via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 20 05:23:11 PDT 2015


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

LG. Yay!


================
Comment at: clang-tidy/modernize/UseDefaultCheck.cpp:59-60
@@ +58,4 @@
+          "= default;");
+  // FIXME: this can generate a -Wpedantic warning if there is a semicolon after
+  // the body: "A() {};" is converted to "A() = default;;"
+}
----------------
Delete the fixme. There already is an extra semicolon in the first case.


http://reviews.llvm.org/D13871





More information about the cfe-commits mailing list