[clang-tools-extra] r239988 - clang-tidy: Remove an unused private field. NFC

Justin Bogner mail at justinbogner.com
Wed Jun 17 21:15:04 PDT 2015


Author: bogner
Date: Wed Jun 17 23:15:04 2015
New Revision: 239988

URL: http://llvm.org/viewvc/llvm-project?rev=239988&view=rev
Log:
clang-tidy: Remove an unused private field. NFC

Clang was warning on this.

Modified:
    clang-tools-extra/trunk/clang-tidy/misc/MacroRepeatedSideEffectsCheck.cpp

Modified: clang-tools-extra/trunk/clang-tidy/misc/MacroRepeatedSideEffectsCheck.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy/misc/MacroRepeatedSideEffectsCheck.cpp?rev=239988&r1=239987&r2=239988&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-tidy/misc/MacroRepeatedSideEffectsCheck.cpp (original)
+++ clang-tools-extra/trunk/clang-tidy/misc/MacroRepeatedSideEffectsCheck.cpp Wed Jun 17 23:15:04 2015
@@ -20,16 +20,14 @@ namespace misc {
 namespace {
 class MacroRepeatedPPCallbacks : public PPCallbacks {
 public:
-  MacroRepeatedPPCallbacks(ClangTidyCheck &Check, SourceManager &SM,
-                           Preprocessor &PP)
-      : Check(Check), SM(SM), PP(PP) {}
+  MacroRepeatedPPCallbacks(ClangTidyCheck &Check, Preprocessor &PP)
+      : Check(Check), PP(PP) {}
 
   void MacroExpands(const Token &MacroNameTok, const MacroDefinition &MD,
                     SourceRange Range, const MacroArgs *Args) override;
 
 private:
   ClangTidyCheck &Check;
-  SourceManager &SM;
   Preprocessor &PP;
 
   unsigned CountArgumentExpansions(const MacroInfo *MI,
@@ -135,7 +133,7 @@ void MacroRepeatedSideEffectsCheck::regi
     CompilerInstance &Compiler) {
   Compiler.getPreprocessor().addPPCallbacks(
       ::llvm::make_unique<MacroRepeatedPPCallbacks>(
-          *this, Compiler.getSourceManager(), Compiler.getPreprocessor()));
+          *this, Compiler.getPreprocessor()));
 }
 
 } // namespace misc





More information about the cfe-commits mailing list