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


================
Comment at: cfe/trunk/lib/Format/Format.cpp:1544
+        return true;
+      for (auto ChildLine : Line->Children) {
+        if (LineContainsObjCCode(*ChildLine))
----------------
benhamilton wrote:
> djasper wrote:
> > 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.
> Oh, of course. I'll send a follow-up.
Follow-up in D44831.


Repository:
  rL LLVM

https://reviews.llvm.org/D44790



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to