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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits