diff options
author | hjk <hjk@qt.io> | 2018-04-04 18:25:23 +0200 |
---|---|---|
committer | hjk <hjk@qt.io> | 2018-04-06 12:55:23 +0000 |
commit | cf4dbb4bb628c4d51ae866b278a5d0733674c129 (patch) | |
tree | 84ecd3c4f841192460142b507bd1e4923b53017c /src/plugins/clangcodemodel/clangfixitoperation.cpp | |
parent | 4a0bbed560a53b644d8d3c9a9db218054582bd89 (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.cpp | 16 |
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; |