clang_complete icon indicating copy to clipboard operation
clang_complete copied to clipboard

Show brief comments (updated patches)

Open tobiasgrosser opened this issue 10 years ago • 4 comments

tobiasgrosser avatar Mar 06 '14 11:03 tobiasgrosser

This is a reworked version of #199

tobiasgrosser avatar Mar 06 '14 11:03 tobiasgrosser

Can you add a comment on toVimRepr to state that pyeval can be used instead? As far as I remember this was added "recently" and certain popular distribution didn't package a newer enough vim.

Rest looks great!

xavierd avatar Mar 08 '14 03:03 xavierd

On 03/08/2014 04:43 AM, Xavier Deguillard wrote:

Can you add a comment on toVimRepr to state that pyeval can be used instead? As far as I remember this was added "recently" and certain popular distribution didn't package a newer enough vim.

I don't get your comment. Something like:

"We can not currently use pyeval, as it is not yet available in common vim installations. However, this functionality could be replaced by pyeval at a later point."

So are you saying pyeval would always do the right conversions?

Tobias

tobiasgrosser avatar Mar 08 '14 08:03 tobiasgrosser

Yes, I believe pyeval can be used instead of the toVimRepr function yes, and should also be much faster.

xavierd avatar Mar 09 '14 05:03 xavierd