Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Very slow to update for long s-exprs #43

Open
metasoarous opened this issue Dec 29, 2017 · 6 comments
Open

Very slow to update for long s-exprs #43

metasoarous opened this issue Dec 29, 2017 · 6 comments

Comments

@metasoarous
Copy link
Contributor

98% of the time, vim-parinfer works great for me, but for code with very long top level forms (long defrecords for instance), it tends to be pretty horrible (well nigh unusable, sometimes forcing me to switch to Cursive). Other parinfer-capable editors I've used don't seem to have this problem, so I'm assuming the issue isn't inherent to the algorithm, but instead the implementation.

Is there a chance #35 might help us here?

@bhurlow
Copy link
Owner

bhurlow commented Dec 29, 2017

@metasoarous yea, noticing the same thing. The VimL implementation is very very slow, so I think #35 would probably be a big improvement

@bhurlow
Copy link
Owner

bhurlow commented Dec 29, 2017

In the first release I actually sent the buffer over the wire to a node.js server running the parinfer js lib and that was also considerably quicker

@bhurlow
Copy link
Owner

bhurlow commented Dec 29, 2017

@metasoarous are you using neovim or regular vim?

@metasoarous
Copy link
Contributor Author

Regular vim.

@eraserhd
Copy link
Contributor

Hi! I ported parinfer to Rust, which Vim can call through libcall(). It's much faster, doesn't require Neovim, and can probably be improved, speed-wise. It might be beta quality, though.

See here: https://github.com/eraserhd/parinfer-rust

@bhurlow
Copy link
Owner

bhurlow commented Mar 14, 2018

@eraserhd can't wait to try it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants