From: Manuel Klimek Date: Fri, 20 Apr 2012 14:07:01 +0000 (+0000) Subject: No need to put the SourceManager in with the ASTContext, as the ASTContext X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=dab28942c4ebd04e279535207c366faedea260a6;p=clang No need to put the SourceManager in with the ASTContext, as the ASTContext already contains the SourceManager. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@155198 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/unittests/Tooling/RecursiveASTVisitorTest.cpp b/unittests/Tooling/RecursiveASTVisitorTest.cpp index d749f76a22..d7dad2738a 100644 --- a/unittests/Tooling/RecursiveASTVisitorTest.cpp +++ b/unittests/Tooling/RecursiveASTVisitorTest.cpp @@ -38,7 +38,6 @@ public: protected: clang::ASTContext *Context; - clang::SourceManager *SM; private: class FindConsumer : public clang::ASTConsumer { @@ -59,7 +58,6 @@ private: virtual clang::ASTConsumer* CreateASTConsumer( clang::CompilerInstance& compiler, llvm::StringRef dummy) { - Visitor->SM = &compiler.getSourceManager(); Visitor->Context = &compiler.getASTContext(); /// TestConsumer will be deleted by the framework calling us. return new FindConsumer(Visitor); @@ -116,7 +114,7 @@ protected: // If we did not match, record information about partial matches. llvm::raw_string_ostream Stream(PartialMatches); Stream << ", partial match: \"" << Name << "\" at "; - Location.print(Stream, *this->SM); + Location.print(Stream, this->Context->getSourceManager()); } }