Re: [PATCH] D23544: [clang-tidy] readability-implicit-bool-cast forgets to store its options.

[Alexander Kornienko via cfe-commits]

alexfh added inline comments.

================
Comment at: clang-tidy/readability/ImplicitBoolCastCheck.h:25
@@ -24,3 +24,3 @@
 public:
   ImplicitBoolCastCheck(StringRef Name, ClangTidyContext *Context)
       : ClangTidyCheck(Name, Context),
----------------
BTW, could you move the constructor body to the .cpp file? It's more 
convenient, when both reading and storing options are placed together.


https://reviews.llvm.org/D23544



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits