Re: [PATCH] D20362: [clang-format] Make formatReplacements() also sort #includes.

2016-05-18 Thread Eric Liu via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL269924: [clang-format] Make formatReplacements() also sort #includes. (authored by ioeric). Changed prior to commit: http://reviews.llvm.org/D20362?vs=57601&id=57612#toc Repository: rL LLVM http://r

Re: [PATCH] D20362: [clang-format] Make formatReplacements() also sort #includes.

2016-05-18 Thread Daniel Jasper via cfe-commits
djasper accepted this revision. djasper added a comment. This revision is now accepted and ready to land. Looks good. I am not even certain whether we want to keep a separate sortIncludes in the interface at all, but as this doesn't change the public API, it doesn't hurt. http://reviews.llvm.o

[PATCH] D20362: [clang-format] Make formatReplacements() also sort #includes.

2016-05-18 Thread Eric Liu via cfe-commits
ioeric created this revision. ioeric added reviewers: bkramer, djasper. ioeric added a subscriber: cfe-commits. Herald added a subscriber: klimek. [clang-format] Make formatReplacements() also sort #includes. http://reviews.llvm.org/D20362 Files: lib/Format/Format.cpp unittests/Format/Format