[PATCH] D20964: [clang-tidy] Add modernize-use-emplace

Piotr Padlewski via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 3 08:22:25 PDT 2016


Prazek marked 4 inline comments as done.

================
Comment at: clang-tidy/modernize/UseEmplaceCheck.cpp:29
@@ +28,3 @@
+  auto callPushBack =
+      cxxMemberCallExpr(hasDeclaration(functionDecl(hasName(PushBackName))),
+                        on(hasType(cxxRecordDecl(hasName(VectorName)))))
----------------
I don't think so. In all checks people use lowerCamelCase, because it looks like a real matchers


http://reviews.llvm.org/D20964





More information about the cfe-commits mailing list