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

vr is extermely slow in Windows7 #52

Open
pavlo-alkhimov opened this issue Oct 6, 2017 · 5 comments
Open

vr is extermely slow in Windows7 #52

pavlo-alkhimov opened this issue Oct 6, 2017 · 5 comments

Comments

@pavlo-alkhimov
Copy link

pavlo-alkhimov commented Oct 6, 2017

Using profiler-start/profiler-report I figured out, that visual-regexp is very slow in Windows. In Linux this problem is not available.
GNU Emacs 25.3.1 (x86_64-w64-mingw32)
of 2017-09-12
visual-regexp 20161017.1713 from elpa

@benma
Copy link
Owner

benma commented Oct 6, 2017

vr starts a Python process for every keystroke, maybe Windows is less efficient with that? Maybe a long running Python process could be used instead.

Happy for anyone using Windows to analyze this and make a PR. Sorry I can't be of more help, I don't have access to any Windows machine.

@pavlo-alkhimov
Copy link
Author

For those who are interested: http://python.readthedocs.io/en/v2.7.2/faq/windows.html#why-does-python-sometimes-take-so-long-to-start
It seems to be exactly my case as I use a company PC which checks if every executed program is permitted to run or not.

@net
Copy link

net commented Oct 1, 2018

Only visual-regexp-steroids.el starts Python, correct?

Should these hooks be added when vr is entered and removed when it is ended?

(add-hook 'after-change-functions 'vr--after-change)
(add-hook 'minibuffer-setup-hook 'vr--minibuffer-setup)

@benma
Copy link
Owner

benma commented Oct 2, 2018

Yes, correct.

The hooks don't do anything when vr is not used actively, so it shouldn't be noticeable.

@benma
Copy link
Owner

benma commented Apr 14, 2019

Should these hooks be added when vr is entered and removed when it is ended?

Finally done in 3e3ed81

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