[PATCH] D44790: [clang-format] Fix ObjC style guesser to also iterate over child lines

Ben Hamilton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 22 10:33:59 PDT 2018


benhamilton marked an inline comment as done.
benhamilton added inline comments.


================
Comment at: lib/Format/Format.cpp:1517
 
-    for (auto &Line : AnnotatedLines) {
-      for (FormatToken *FormatTok = Line->First; FormatTok;
+    auto CheckLineTokens = [&Keywords](const AnnotatedLine &Line) {
+      for (const FormatToken *FormatTok = Line.First; FormatTok;
----------------
djasper wrote:
> Maybe choose a name that indicates what the bool result value means, e.g. LinesContainObjCCode or something.
Done, thanks!


Repository:
  rC Clang

https://reviews.llvm.org/D44790





More information about the cfe-commits mailing list