This revision was automatically updated to reflect the committed changes.
Closed by commit rL294459: [clang-tidy] Supresses misc-move-constructor-init
warning for const fields. (authored by alexfh).
Changed prior to commit:
https://reviews.llvm.org/D28973?vs=87484&id=87655#toc
Repository:
rL
alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.
LG. I'll commit the patch for you.
https://reviews.llvm.org/D28973
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/
lethalantidote updated this revision to Diff 87484.
lethalantidote marked an inline comment as done.
lethalantidote added a comment.
- Addresses alexfh's comments.
https://reviews.llvm.org/D28973
Files:
clang-tools-extra/clang-tidy/misc/MoveConstructorInitCheck.cpp
clang-tools-extra/test/cl
alexfh requested changes to this revision.
alexfh added inline comments.
This revision now requires changes to proceed.
Comment at: clang-tools-extra/test/clang-tidy/misc-move-constructor-init.cpp:88
+struct O {
+ O(O&& other) : b(other.b) {} // ok
+ const B b;
lethalantidote created this revision.
Supresses misc-move-constructor init warning for const fields.
https://reviews.llvm.org/D28973
Files:
clang-tools-extra/clang-tidy/misc/MoveConstructorInitCheck.cpp
clang-tools-extra/test/clang-tidy/misc-move-constructor-init.cpp
Index: clang-tools-ex
