[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
Fri Mar 23 05:48:47 PDT 2018
djasper added inline comments.
================
Comment at: cfe/trunk/lib/Format/Format.cpp:1544
+ return true;
+ for (auto ChildLine : Line->Children) {
+ if (LineContainsObjCCode(*ChildLine))
----------------
Sorry that I missed this in the original review. But doesn't this still fail for Children of Child lines, etc.? I think this probably needs to be fully recursive.
Repository:
rL LLVM
https://reviews.llvm.org/D44790
More information about the cfe-commits
mailing list