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

Reply via email to