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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to