Re: [PATCH] D20816: [include-fixer] use clang-format cleaner to insert header.

2016-05-31 Thread Haojian Wu via cfe-commits
hokein added inline comments. Comment at: include-fixer/IncludeFixer.cpp:29 @@ -28,3 +28,3 @@ class Action; The forward declaration can be removed too. Repository: rL LLVM http://reviews.llvm.org/D20816 ___

Re: [PATCH] D20816: [include-fixer] use clang-format cleaner to insert header.

2016-05-31 Thread Eric Liu via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL271287: [include-fixer] use clang-format cleaner to insert header. (authored by ioeric). Changed prior to commit: http://reviews.llvm.org/D20816?vs=59067&id=59069#toc Repository: rL LLVM http://revi

Re: [PATCH] D20816: [include-fixer] use clang-format cleaner to insert header.

2016-05-31 Thread Benjamin Kramer via cfe-commits
bkramer accepted this revision. bkramer added a comment. This revision is now accepted and ready to land. Look at all this annoying preprocessor code going away! I love it. http://reviews.llvm.org/D20816 ___ cfe-commits mailing list cfe-commits@list