Re: [PATCH] D20370: [include-fixer] Sort headers after inserting new headers.

2016-05-19 Thread Eric Liu via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL270031: [include-fixer] Sort headers after inserting new headers. (authored by ioeric). Changed prior to commit: http://reviews.llvm.org/D20370?vs=57745&id=57746#toc Repository: rL LLVM http://revie

Re: [PATCH] D20370: [include-fixer] Sort headers after inserting new headers.

2016-05-19 Thread Benjamin Kramer via cfe-commits
bkramer accepted this revision. bkramer added a comment. This revision is now accepted and ready to land. lgtm http://reviews.llvm.org/D20370 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-

Re: [PATCH] D20370: [include-fixer] Sort headers after inserting new headers.

2016-05-19 Thread Eric Liu via cfe-commits
ioeric updated this revision to Diff 57745. ioeric added a comment. - Passed Headers into IncludeFixerActionFactory so that we can know which headers are added. http://reviews.llvm.org/D20370 Files: include-fixer/CMakeLists.txt include-fixer/IncludeFixer.cpp include-fixer/IncludeFixer.h