[PATCH] Add readability-remove-void-arg check to clang-tidy

Richard legalize at xmission.com
Mon Mar 2 22:11:27 PST 2015


================
Comment at: clang-tidy/readability/RemoveVoidArg.cpp:164
@@ +163,3 @@
+    const MatchFinder::MatchResult &Result, SourceLocation StartLoc,
+    const std::string &DeclText, const StringRef GrammarLocation) {
+  clang::Lexer PrototypeLexer(StartLoc, Result.Context->getLangOpts(),
----------------
alexfh wrote:
> s/const std::string&/StringRef/, s/const StringRef/StringRef/
DeclText needs to stay a std::string or lexing fails.

http://reviews.llvm.org/D7639

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list