[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