[PATCH] D20621: [include-fixer] Create a mode in vim integration to show multiple potential headers.

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Mon May 30 03:10:47 PDT 2016


hokein added a comment.

In http://reviews.llvm.org/D20621#439447, @bkramer wrote:

> Can you add some lit tests for the various command line modes clang-include-fixer has now. We can't reasonably test the vim integration but we can tests the bits it's composed of.


Done.


================
Comment at: include-fixer/tool/clang-include-fixer.py:49
@@ +48,3 @@
+                                                  default_choice_index)
+  return int(vim.eval(to_eval));
+
----------------
ioeric wrote:
> Maybe handle cases where `to_eval` is not a valid index?
vim can handle this for us. 


http://reviews.llvm.org/D20621





More information about the cfe-commits mailing list