[PATCH] D141868: [Clang] [Sema] Removed a fix-it for system headers

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 18 13:43:08 PST 2023


NoQ added inline comments.


================
Comment at: clang/lib/Sema/SemaOverload.cpp:10923-10927
+    for (std::vector<FixItHint>::iterator HI = Cand->Fix.Hints.begin(),
+                                          HE = Cand->Fix.Hints.end();
+         HI != HE; ++HI)
+        FDiag << *HI;
+  }
----------------
NoQ wrote:
> Since you're reformatting anyway, maybe use a range-based for-loop?
(or `for_each`)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D141868/new/

https://reviews.llvm.org/D141868



More information about the cfe-commits mailing list