From: Jordan Rose Date: Fri, 28 Sep 2012 22:29:02 +0000 (+0000) Subject: Fix buildbots by not using a template from another namespace. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=20441c59a00afb949e53121c55f8a58da17ba79e;p=clang Fix buildbots by not using a template from another namespace. No need to specialize BeforeThanCompare for a comparator that's only going to be used once. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@164859 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Sema/AnalysisBasedWarnings.cpp b/lib/Sema/AnalysisBasedWarnings.cpp index ec500a06c3..ad5c739037 100644 --- a/lib/Sema/AnalysisBasedWarnings.cpp +++ b/lib/Sema/AnalysisBasedWarnings.cpp @@ -936,23 +936,22 @@ static void DiagnoseSwitchLabelsFallthrough(Sema &S, AnalysisDeclContext &AC, } namespace { - typedef std::pair - StmtUsesPair; -} +typedef std::pair + StmtUsesPair; -template<> -class BeforeThanCompare { +class StmtUseSorter { const SourceManager &SM; public: - explicit BeforeThanCompare(const SourceManager &SM) : SM(SM) { } + explicit StmtUseSorter(const SourceManager &SM) : SM(SM) { } bool operator()(const StmtUsesPair &LHS, const StmtUsesPair &RHS) { return SM.isBeforeInTranslationUnit(LHS.first->getLocStart(), RHS.first->getLocStart()); } }; +} static void diagnoseRepeatedUseOfWeak(Sema &S, @@ -991,7 +990,7 @@ static void diagnoseRepeatedUseOfWeak(Sema &S, // Sort by first use so that we emit the warnings in a deterministic order. std::sort(UsesByStmt.begin(), UsesByStmt.end(), - BeforeThanCompare(S.getSourceManager())); + StmtUseSorter(S.getSourceManager())); // Classify the current code body for better warning text. // This enum should stay in sync with the cases in