[PATCH] D29262: Fixes to modernize-use-using

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 9 07:49:01 PST 2017


alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.

LG with a few comments.



================
Comment at: clang-tidy/modernize/UseUsingCheck.h:24
 class UseUsingCheck : public ClangTidyCheck {
+
 public:
----------------
nit: empty line


================
Comment at: clang-tidy/modernize/UseUsingCheck.h:32
+private:
+  const bool MacroWarning;
 };
----------------
nit: `MacroWarning` doesn't sound right to me. Maybe `WarnInMacros` or even `IgnoreMacros` (defaulting to `true`)?


================
Comment at: test/clang-tidy/modernize-use-using.cpp:154
+// CHECK-MESSAGES: :[[@LINE-1]]:3: warning: use 'using' instead of 'typedef'
+// CHECK-FIXES: using FuncType = my_cclass
+}
----------------
Why no trailing semicolon?


https://reviews.llvm.org/D29262





More information about the cfe-commits mailing list