[PATCH] D22351: [include-fixer] Move cursor to #include line in vim after inserting a missing header.
Daniel Jasper via cfe-commits
cfe-commits at lists.llvm.org
Fri Jul 15 00:34:23 PDT 2016
djasper added a comment.
FWIW, I think I'd like this behavior as it shows me what is actually being done. I can undo/redo to double-check or C-O to get back to where I was. Of course making it configurable seems like the right thing to do.
================
Comment at: include-fixer/tool/clang-include-fixer.py:102
@@ +101,3 @@
+ # Set cursor to the #include line.
+ include_header = "#include " + header["HeaderInfos"][0]["Header"]
+ line_num = lines.index(include_header) + 1
----------------
Seems seems very hacky. Can't you extract it out of the diff sequence above?
https://reviews.llvm.org/D22351
More information about the cfe-commits
mailing list