summaryrefslogtreecommitdiffstats
path: root/clang-tools-extra/clang-tidy/bugprone/MultipleStatementMacroCheck.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'clang-tools-extra/clang-tidy/bugprone/MultipleStatementMacroCheck.cpp')
-rw-r--r--clang-tools-extra/clang-tidy/bugprone/MultipleStatementMacroCheck.cpp13
1 files changed, 7 insertions, 6 deletions
diff --git a/clang-tools-extra/clang-tidy/bugprone/MultipleStatementMacroCheck.cpp b/clang-tools-extra/clang-tidy/bugprone/MultipleStatementMacroCheck.cpp
index 9fa2f43c1f7..942c8752f35 100644
--- a/clang-tools-extra/clang-tidy/bugprone/MultipleStatementMacroCheck.cpp
+++ b/clang-tools-extra/clang-tidy/bugprone/MultipleStatementMacroCheck.cpp
@@ -38,7 +38,7 @@ const Stmt *nextStmt(const MatchFinder::MatchResult &Result, const Stmt *S) {
return nextStmt(Result, Parent);
}
-using ExpansionRanges = std::vector<std::pair<SourceLocation, SourceLocation>>;
+using ExpansionRanges = std::vector<SourceRange>;
/// \bried Get all the macro expansion ranges related to `Loc`.
///
@@ -47,8 +47,9 @@ ExpansionRanges getExpansionRanges(SourceLocation Loc,
const MatchFinder::MatchResult &Result) {
ExpansionRanges Locs;
while (Loc.isMacroID()) {
- Locs.push_back(Result.SourceManager->getImmediateExpansionRange(Loc));
- Loc = Locs.back().first;
+ Locs.push_back(
+ Result.SourceManager->getImmediateExpansionRange(Loc).getAsRange());
+ Loc = Locs.back().getBegin();
}
return Locs;
}
@@ -96,9 +97,9 @@ void MultipleStatementMacroCheck::check(
InnerRanges.back() != NextRanges.back())
return;
- diag(InnerRanges.back().first, "multiple statement macro used without "
- "braces; some statements will be "
- "unconditionally executed");
+ diag(InnerRanges.back().getBegin(), "multiple statement macro used without "
+ "braces; some statements will be "
+ "unconditionally executed");
}
} // namespace bugprone
OpenPOWER on IntegriCloud