Re: [PATCH] D24791: Use checktime when reloading vim buffer after applying clang-rename

2016-09-26 Thread Kirill Bobyrev via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL282388: [clang-rename] Use checktime when reloading vim buffer after applying clang… (authored by omtcyfz). Changed prior to commit: https://reviews.llvm.org/D24791?vs=72011&id=72443#toc Repository:

Re: [PATCH] D24791: Use checktime when reloading vim buffer after applying clang-rename

2016-09-25 Thread Kai Wolf via cfe-commits
NewProggie added a comment. In https://reviews.llvm.org/D24791#551662, @omtcyfz wrote: > @NewProggie do you have commit access or do you want me to land this patch > for you? I don't have commit access. Please, land it for me. Glad, you've accepted my patch. https://reviews.llvm.org/D24791

Re: [PATCH] D24791: Use checktime when reloading vim buffer after applying clang-rename

2016-09-24 Thread Kirill Bobyrev via cfe-commits
omtcyfz added a comment. @NewProggie do you have commit access or do you want me to land this patch for you? https://reviews.llvm.org/D24791 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-

Re: [PATCH] D24791: Use checktime when reloading vim buffer after applying clang-rename

2016-09-24 Thread Kirill Bobyrev via cfe-commits
omtcyfz accepted this revision. omtcyfz added a comment. This revision is now accepted and ready to land. Sorry for a late response. Yes, this solution is way better. Honestly I just didn't know how to achieve that; not a Vim pro here :) Thank you very much! LGTM. https://reviews.llvm.org/D2

[PATCH] D24791: Use checktime when reloading vim buffer after applying clang-rename

2016-09-21 Thread Kai Wolf via cfe-commits
NewProggie created this revision. NewProggie added a subscriber: cfe-commits. After applying `clang-rename` to a vim buffer (using `clang-rename.py` as part of the vim integration) the buffer gets reloaded using `bufdo`. This solution is suboptimal, since syntax highlighting is turned off for pe