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.

Sign up for GitHub

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

Jump to bottom

Speed #2

Open
SimonSapin opened this issue Mar 23, 2016 · 2 comments · May be fixed by #11
Open

Speed #2

SimonSapin opened this issue Mar 23, 2016 · 2 comments · May be fixed by #11

Comments

@SimonSapin
Copy link
Contributor

Thank you for making this, I like it!

A minor downside is that I see some noticeable lag at each keystroke when typing, especially if my CPU cores are busy (even though I’m running Cargo under nice).

Do you know if gedit’s UI is blocked while racer is running? If so, making things asynchronous would maybe reduce the perception of lag.

Or do you have other ideas for what could make things faster? Maybe having a long-running racer process rather than spawning a new one for every keystroke?
@isamert
Copy link
Owner

isamert commented Mar 23, 2016

Other than spawning new racer proccess everytime, it saves a new temp file at each keystroke. That can also cause the lag but it's only way to communicate with racer. So as you said, running racer async or using a racer daemon maybe reduce the lag. I'll look into that.

And thanks for your interest!

@SimonSapin
Copy link
Contributor Author

racer can probably be modified to accept input on stdin rather than a (temporary) file, but I don’t know if that is a bottleneck.

Let me know if I can help.

isamert added a commit that referenced this issue Apr 14, 2016
@isamert
improves #2
cbff954
@egormanga egormanga linked a pull request Dec 5, 2022 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Updated for gedit 43 egormanga/gracer
3 participants
@SimonSapin @isamert and others