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

Daniel Jasper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 22 10:29:23 PDT 2018


djasper accepted this revision.
djasper added a comment.
This revision is now accepted and ready to land.

Looks good, thank you!



================
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;
----------------
Maybe choose a name that indicates what the bool result value means, e.g. LinesContainObjCCode or something.


Repository:
  rC Clang

https://reviews.llvm.org/D44790





More information about the cfe-commits mailing list