Re: [PATCH] D20232: [include-fixer] Simplify unittest code.

2016-05-13 Thread Haojian Wu via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL269427: [include-fixer] Simplify unittest code. (authored by hokein). Changed prior to commit: http://reviews.llvm.org/D20232?vs=57193&id=57194#toc Repository: rL LLVM http://reviews.llvm.org/D20232

Re: [PATCH] D20232: [include-fixer] Simplify unittest code.

2016-05-13 Thread Haojian Wu via cfe-commits
hokein updated this revision to Diff 57193. hokein updated the summary for this revision. hokein added a comment. Fix clang-include-fixer build. http://reviews.llvm.org/D20232 Files: include-fixer/InMemorySymbolIndex.cpp include-fixer/InMemorySymbolIndex.h include-fixer/find-all-symbols/F

Re: [PATCH] D20232: [include-fixer] Simplify unittest code.

2016-05-13 Thread Benjamin Kramer via cfe-commits
bkramer added a comment. So s/flexable/flexible/ and commit :) Repository: rL LLVM http://reviews.llvm.org/D20232 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D20232: [include-fixer] Simplify unittest code.

2016-05-13 Thread Haojian Wu via cfe-commits
hokein added a comment. In http://reviews.llvm.org/D20232#429348, @bkramer wrote: > What's a flexable (in the description)? Code LG. I updated the description now. Repository: rL LLVM http://reviews.llvm.org/D20232 ___ cfe-commits mailing list

Re: [PATCH] D20232: [include-fixer] Simplify unittest code.

2016-05-13 Thread Benjamin Kramer via cfe-commits
bkramer accepted this revision. bkramer added a comment. This revision is now accepted and ready to land. What's a flexable (in the description)? Code LG. Repository: rL LLVM http://reviews.llvm.org/D20232 ___ cfe-commits mailing list cfe-commits