[clang-tools-extra] r284589 - Don't copy replacements in for-range loop. NFC.

Benjamin Kramer via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 19 06:50:18 PDT 2016


Author: d0k
Date: Wed Oct 19 08:50:17 2016
New Revision: 284589

URL: http://llvm.org/viewvc/llvm-project?rev=284589&view=rev
Log:
Don't copy replacements in for-range loop. NFC.

Modified:
    clang-tools-extra/trunk/include-fixer/tool/ClangIncludeFixer.cpp

Modified: clang-tools-extra/trunk/include-fixer/tool/ClangIncludeFixer.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/include-fixer/tool/ClangIncludeFixer.cpp?rev=284589&r1=284588&r2=284589&view=diff
==============================================================================
--- clang-tools-extra/trunk/include-fixer/tool/ClangIncludeFixer.cpp (original)
+++ clang-tools-extra/trunk/include-fixer/tool/ClangIncludeFixer.cpp Wed Oct 19 08:50:17 2016
@@ -416,7 +416,7 @@ int includeFixerMain(int argc, const cha
 
   // Write replacements to disk.
   Rewriter Rewrites(SM, LangOptions());
-  for (const auto Replacement : FixerReplacements) {
+  for (const auto &Replacement : FixerReplacements) {
     if (!tooling::applyAllReplacements(Replacement, Rewrites)) {
       llvm::errs() << "Failed to apply replacements.\n";
       return 1;




More information about the cfe-commits mailing list