[PATCH] D18919: [Clang-tidy] Add check "modernize use using"

Etienne Bergeron via cfe-commits cfe-commits at lists.llvm.org
Thu May 12 13:30:01 PDT 2016


etienneb added a subscriber: etienneb.

================
Comment at: clang-tidy/modernize/UseUsingCheck.cpp:32
@@ +31,3 @@
+
+  diag(MatchedDecl->getLocStart(), "use using instead of typedef")
+      << FixItHint::CreateReplacement(
----------------
I think 'using' should be quote.
This message is hard to read and understand from a user point of view.

================
Comment at: test/clang-tidy/modernize-use-using.cpp:48
@@ +47,3 @@
+};
+
+using balba = long long;
----------------
Could you add test with macro:

#define CODE \
  typedef int INT; \
  [....]

CODE;


Repository:
  rL LLVM

http://reviews.llvm.org/D18919





More information about the cfe-commits mailing list