Re: [PATCH] D23274: Add an option to clang-format to remove duplicate headers.

2016-08-10 Thread Daniel Jasper via cfe-commits
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

Re: [PATCH] D23274: Add an option to clang-format to remove duplicate headers.

2016-08-08 Thread Eric Liu via cfe-commits
ioeric updated this revision to Diff 67191. ioeric added a comment. - Check code that is not in affected range is not reformatted in lit test. https://reviews.llvm.org/D23274 Files: include/clang/Format/Format.h lib/Format/Format.cpp test/Format/remove-duplicate-includes.cpp tools/clang