[PATCH] D16962: clang-tidy: avoid std::bind

Jonathan B Coe via cfe-commits cfe-commits at lists.llvm.org
Tue May 3 04:45:54 PDT 2016


jbcoe added inline comments.

================
Comment at: clang-tidy/readability/ReadabilityTidyModule.cpp:37
@@ -35,1 +36,3 @@
+    CheckFactories.registerCheck<AvoidStdBindCheck>(
+        "readability-avoid-std-bind");
     CheckFactories.registerCheck<BracesAroundStatementsCheck>(
----------------
aaron.ballman wrote:
> I kind of wonder if this should be in modernize instead of readability? Tough call, and I'm fine with either place, just wanted to pose the question.
Given that lambdas and `std::bind` both arrived in C++11, `modernize` seems a misnomer.
I think `readability` is better fit.


http://reviews.llvm.org/D16962





More information about the cfe-commits mailing list