diff options
author | Marek Kurdej <marek.kurdej+llvm.org@gmail.com> | 2022-01-07 15:21:57 +0100 |
---|---|---|
committer | Marek Kurdej <marek.kurdej+llvm.org@gmail.com> | 2022-01-07 16:06:11 +0100 |
commit | 4681ae9353ed89d28a95f4d07e8050f8772ae724 (patch) | |
tree | 931f81b98d07e5df4ec7e6197cabb36df981e2fd /clang/lib/Format/QualifierAlignmentFixer.cpp | |
parent | c747b2c0b09e8a97521bffc8bbfe17a4670dd534 (diff) | |
download | llvm-4681ae9353ed89d28a95f4d07e8050f8772ae724.zip llvm-4681ae9353ed89d28a95f4d07e8050f8772ae724.tar.gz llvm-4681ae9353ed89d28a95f4d07e8050f8772ae724.tar.bz2 |
[clang-format] Use range-for loops. NFC.
* Avoid if check on every element of the loop when printing symbols.
Diffstat (limited to 'clang/lib/Format/QualifierAlignmentFixer.cpp')
-rw-r--r-- | clang/lib/Format/QualifierAlignmentFixer.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/clang/lib/Format/QualifierAlignmentFixer.cpp b/clang/lib/Format/QualifierAlignmentFixer.cpp index ec19a38..a53db5d 100644 --- a/clang/lib/Format/QualifierAlignmentFixer.cpp +++ b/clang/lib/Format/QualifierAlignmentFixer.cpp @@ -88,11 +88,11 @@ std::pair<tooling::Replacements, unsigned> QualifierAlignmentFixer::analyze( // Don't make replacements that replace nothing. tooling::Replacements NonNoOpFixes; - for (auto I = Fixes.begin(), E = Fixes.end(); I != E; ++I) { - StringRef OriginalCode = Code.substr(I->getOffset(), I->getLength()); + for (const tooling::Replacement &Fix : Fixes) { + StringRef OriginalCode = Code.substr(Fix.getOffset(), Fix.getLength()); - if (!OriginalCode.equals(I->getReplacementText())) { - auto Err = NonNoOpFixes.add(*I); + if (!OriginalCode.equals(Fix.getReplacementText())) { + auto Err = NonNoOpFixes.add(Fix); if (Err) llvm::errs() << "Error adding replacements : " << llvm::toString(std::move(Err)) << "\n"; @@ -396,9 +396,9 @@ LeftRightQualifierAlignmentFixer::analyze( tok::TokenKind QualifierToken = getTokenFromQualifier(Qualifier); assert(QualifierToken != tok::identifier && "Unrecognised Qualifier"); - for (size_t I = 0, E = AnnotatedLines.size(); I != E; ++I) { - FormatToken *First = AnnotatedLines[I]->First; - const auto *Last = AnnotatedLines[I]->Last; + for (AnnotatedLine *Line : AnnotatedLines) { + FormatToken *First = Line->First; + const auto *Last = Line->Last; for (const auto *Tok = First; Tok && Tok != Last && Tok->Next; Tok = Tok->Next) { |