lyuts / vim-rtags

Vim bindings for rtags, llvm/clang based c++ code indexer.
BSD 2-Clause "Simplified" License
282 stars 56 forks source link

Plugin should not send buffer as --unsaved-file unless needed #38

Closed jmacdonagh closed 8 years ago

jmacdonagh commented 8 years ago

Sending the buffer contents using --unsaved-file is unnecessary if the buffer has no unsaved changes. rdm tracks and re-indexes any saved files.

I've been using vim-rtags to browse a very large source code base (no changes, being made, just read only). Any time I follow links or find references it's delayed as rdm has to reindex the potentially large file (way worse when it's a header file used by many source files).

It would be nice if this were configurable. An option that takes 3 states:

Don't know if you have any thoughts on this. I was going to implement this, but I was wondering if you had other plans.

bagnose commented 8 years ago

You may be interested in https://github.com/acres-com-au/vim-rtags/commit/2871d868df58d00ea58ac8e539dd30c15da9456a

jmacdonagh commented 8 years ago

@bagnose, that looks like exactly what I was looking for. I'll check with the author and see if they'll put in a pull request. Great find!

jmacdonagh commented 8 years ago

@quixotique, are you able to create separate pull requests for the two useful commits commits you've made to vim-rtags (specifically https://github.com/acres-com-au/vim-rtags/commit/2871d868df58d00ea58ac8e539dd30c15da9456a)?

quixotique commented 8 years ago

Just created pull request #39, couldn't work out how to make separate requests for the different commits, as I am pressed for time.

jmacdonagh commented 8 years ago

Thanks @quixotique!

lyuts commented 8 years ago

Pull request #39 is now merged.