djasper added inline comments.

================
Comment at: lib/Format/TokenAnnotator.cpp:288
 
+        if (MightBeObjCForRangeLoop) {
+          FormatToken *ForInToken = Left;
----------------
There can be only one ObjCForIn token in any for loop, right? If that's the 
case, can we just remember that in addition to (or instead of) 
MightBeObjCForRangeLoop? That way, we wouldn't have to re-iterate over all the 
tokens here, but could just set this directly.


Repository:
  rC Clang

https://reviews.llvm.org/D43904



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

Reply via email to