[PATCH] D18265: [clang-tidy] New: checker misc-assign-operator-return

Balogh, Ádám via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 30 03:59:15 PDT 2016


baloghadamsoftware marked an inline comment as done.

================
Comment at: clang-tidy/misc/MiscTidyModule.cpp:54
@@ -55,1 +53,3 @@
+    CheckFactories.registerCheck<AssignOperatorCheck>(
+        "misc-assign-operator");
     CheckFactories.registerCheck<BoolPointerImplicitConversionCheck>(
----------------
alexfh wrote:
> Check names usually contain hints at the class of problems they detect (e.g. "misc-inaccurate-erase" or "misc-macro-repeated-side-effects"), while `misc-assign-operator` doesn't serve this purpose and is overall a bit too vague (there's nothing wrong with assign operators per se).
> 
> I wonder whether a more specific name would be more helpful to the users. I'm thinking of something like `misc-assign-operator-conventions`. WDYT?
I have no better idea either. Should I rename it or let us wait for other potential reviewers proposing another name?


http://reviews.llvm.org/D18265





More information about the cfe-commits mailing list