[PATCH] D40288: [clang-format] Add option to group multiple #include blocks when sorting includes

2017-11-26 Thread Krzysztof Kapusta via Phabricator via cfe-commits
KrzysztofKapusta added a comment. @krasimir thanks for the review! I don't have commit rights. Would you push this for me? Repository: rL LLVM https://reviews.llvm.org/D40288 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.l

[PATCH] D40288: [clang-format] Add option to group multiple #include blocks when sorting includes

2017-11-22 Thread Krzysztof Kapusta via Phabricator via cfe-commits
KrzysztofKapusta added inline comments. Comment at: unittests/Format/SortIncludesTest.cpp:357 + "#include \"c.h\"\n", + "a.cc")); +} krasimir wrote: > What is this testing? I changed this to better reflect what I was trying to test

[PATCH] D40288: [clang-format] Add option to group multiple #include blocks when sorting includes

2017-11-22 Thread Krzysztof Kapusta via Phabricator via cfe-commits
KrzysztofKapusta updated this revision to Diff 123924. KrzysztofKapusta added a comment. Addressed all comments. Changed one testcase to better reflect the desired effect. Repository: rL LLVM https://reviews.llvm.org/D40288 Files: docs/ClangFormatStyleOptions.rst include/clang/Format/Fo

[PATCH] D40288: [clang-format] Add option to group multiple #include blocks when sorting includes

2017-11-21 Thread Krzysztof Kapusta via Phabricator via cfe-commits
KrzysztofKapusta created this revision. KrzysztofKapusta added a project: clang-tools-extra. Herald added a subscriber: klimek. This patch allows grouping multiple #include blocks together and sort all includes as one big block. Additionally, sorted includes can be regrouped after sorting based o