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