[PATCH] D23274: Add an option to clang-format to remove duplicate headers.
Daniel Jasper via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 10 01:04:25 PDT 2016
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.
More information about the cfe-commits