aboutsummaryrefslogtreecommitdiffstats
path: root/src/plugins/clangcodemodel/clangfixitoperation.cpp
diff options
context:
space:
mode:
authorhjk <hjk@qt.io>2018-04-04 18:25:23 +0200
committerhjk <hjk@qt.io>2018-04-06 12:55:23 +0000
commitcf4dbb4bb628c4d51ae866b278a5d0733674c129 (patch)
tree84ecd3c4f841192460142b507bd1e4923b53017c /src/plugins/clangcodemodel/clangfixitoperation.cpp
parent4a0bbed560a53b644d8d3c9a9db218054582bd89 (diff)
ClangSupport: Use simpler structures in some cases
The patch is mostly mechanical, but contains also a few spurious changes from values references for some local variables, foreach -> ranged for etc that I coulnd't resist. Change-Id: I58f0bd972546895eb318607cbfbd7ac35caf3f23 Reviewed-by: Ivan Donchevskii <ivan.donchevskii@qt.io>
Diffstat (limited to 'src/plugins/clangcodemodel/clangfixitoperation.cpp')
-rw-r--r--src/plugins/clangcodemodel/clangfixitoperation.cpp16
1 files changed, 8 insertions, 8 deletions
diff --git a/src/plugins/clangcodemodel/clangfixitoperation.cpp b/src/plugins/clangcodemodel/clangfixitoperation.cpp
index f064b8d2778..b123c9af457 100644
--- a/src/plugins/clangcodemodel/clangfixitoperation.cpp
+++ b/src/plugins/clangcodemodel/clangfixitoperation.cpp
@@ -60,8 +60,8 @@ static FileToFixits fixitsPerFile(const QVector<ClangBackEnd::FixItContainer> &f
FileToFixits mapping;
for (const auto &fixItContainer : fixItContainers) {
- const QString rangeStartFilePath = fixItContainer.range().start().filePath().toString();
- const QString rangeEndFilePath = fixItContainer.range().end().filePath().toString();
+ const QString rangeStartFilePath = fixItContainer.range.start.filePath.toString();
+ const QString rangeEndFilePath = fixItContainer.range.end.filePath.toString();
QTC_CHECK(rangeStartFilePath == rangeEndFilePath);
mapping[rangeStartFilePath].append(fixItContainer);
}
@@ -109,12 +109,12 @@ Utils::ChangeSet ClangFixItOperation::toChangeSet(
Utils::ChangeSet changeSet;
for (const auto &fixItContainer : fixItContainers) {
- const auto range = fixItContainer.range();
- const auto start = range.start();
- const auto end = range.end();
- changeSet.replace(refactoringFile.position(start.line(), start.column()),
- refactoringFile.position(end.line(), end.column()),
- fixItContainer.text());
+ const auto &range = fixItContainer.range;
+ const auto &start = range.start;
+ const auto &end = range.end;
+ changeSet.replace(refactoringFile.position(start.line, start.column),
+ refactoringFile.position(end.line, end.column),
+ fixItContainer.text);
}
return changeSet;