[PATCH] D14145: modernize-use-default supports copy constructor and copy-assignment operator.
Alexander Kornienko via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 28 17:13:22 PDT 2015
alexfh added inline comments.
================
Comment at: test/clang-tidy/modernize-use-default.cpp:139
@@ +138,3 @@
+
+struct O {
+ O() {
----------------
Can we choose a different name for it, that does not resemble a digit?
================
Comment at: test/clang-tidy/modernize-use-default.cpp:147
@@ +146,2 @@
+};
+
----------------
Please add a test with a function try block. Something along the lines of:
struct A {};
struct B : public A {
B() try : A() {} catch(...) {}
};
http://reviews.llvm.org/D14145
More information about the cfe-commits
mailing list