djasper added inline comments. ================ Comment at: lib/Format/Format.cpp:1242 @@ -1240,1 +1241,3 @@ }); + // The index of the include on which the cursor is currently put. + unsigned CurrentCursorIndex = UINT_MAX; ---------------- This needs a comment on what it is actually doing (including what the intended behavior is). And that comment should also go into the patch description.
Also, it might be a good idea to pull this out into a function. https://reviews.llvm.org/D23274 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits