Re: [PATCH] D23153: Run clang-format on clang-rename code

2016-08-04 Thread Miklos Vajna via cfe-commits
vmiklos added a comment. Thanks. Yes, I used the Linux-distro-provided clang-format-3.7, I didn't notice that the trunk version now also sorts includes. Repository: rL LLVM https://reviews.llvm.org/D23153 ___ cfe-commits mailing list cfe-commits

Re: [PATCH] D23153: Run clang-format on clang-rename code

2016-08-04 Thread Kirill Bobyrev via cfe-commits
omtcyfz added a comment. Actually few changes `clang-format` would produce weren't there. It's either you used old `clang-format` or you missed them, but there were header reordering issues. Pushed quick fix. Repository: rL LLVM https://reviews.llvm.org/D23153 ___

Re: [PATCH] D23153: Run clang-format on clang-rename code

2016-08-04 Thread Miklos Vajna via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL277702: Run clang-format on clang-rename code (authored by vmiklos). Changed prior to commit: https://reviews.llvm.org/D23153?vs=66766&id=66769#toc Repository: rL LLVM https://reviews.llvm.org/D2315

Re: [PATCH] D23153: Run clang-format on clang-rename code

2016-08-04 Thread Manuel Klimek via cfe-commits
klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. lg https://reviews.llvm.org/D23153 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com

[PATCH] D23153: Run clang-format on clang-rename code

2016-08-04 Thread Miklos Vajna via cfe-commits
vmiklos created this revision. vmiklos added reviewers: klimek, omtcyfz. vmiklos added a subscriber: cfe-commits. So that later commits don't introduce non-functional changes when running clang-format before committing. https://reviews.llvm.org/D23153 Files: clang-rename/RenamingAction.h cla