[PATCH] D22087: [clang-rename] add basic vim integration

Eric Liu via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 7 05:25:29 PDT 2016


ioeric added inline comments.

================
Comment at: clang-rename/tool/clang-rename.py:12
@@ +11,3 @@
+
+    map ,cf :pyf <path-to>/clang-include-fixer.py<cr>
+
----------------
Maybe a different key binding so that it doesn't conflict with include-fixer's suggested key binding? `,cr` for clang-rename maybe? 

================
Comment at: clang-rename/tool/clang-rename.py:33
@@ +32,3 @@
+    if offset < 0:
+        print 'Couldn\'t determine cursor position. Is your file empty?'
+        return
----------------
Redirect error messages to stderr so that they are highlighted in vim?


http://reviews.llvm.org/D22087





More information about the cfe-commits mailing list