-
Notifications
You must be signed in to change notification settings - Fork 646
"Stuck" keystroke while waiting for autocomplete popup #1272
Comments
Update on this: I recently upgraded my Go version and it seems gocode didn't like that very much. I deleted everything under $GOPATH/pkg, reinstalled gocode with However, there does remain an issue with vscode: if |
That scares me
We could kill the process that calls Thoughts? |
I'm thinking that perhaps the issue might be gocode chewing up every single free cycle, slowing my system to a crawl and causing everything to behave oddly. I watched my processors the last time it happened and they all spiked to 100%. However, that still shouldn't cause VSCode to repeat the last keypress as far as I'm concerned. I get the feeling it registers the key down event, gets stuck waiting on gocode, then registers the keyup event. Perhaps it thinks the key is being held? I don't know. Regardless, I think killing the gocode process would be a decent way to approach this. It may even be worth adding a "fix it for me" popup or something like that. Basically it just cleans out the |
A bit more here: I just did Regardless, it's smooth now. I'll just have to manually build once in a while. |
Thanks @tylerb for getting back. |
To be fair, the issue still exist. The key does get stuck while gocode is building. It seems that should not happen regardless of what gocode is doing
…On Nov 1, 2017, 5:21 PM -0600, Ramya Rao ***@***.***>, wrote:
Closed #1272.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Sorry about that. |
Would it be possible for us not to block keyboard events while waiting for gocode to respond? I feel like that would be a better solution than killing the process.
Then again there could be details there of which I am not aware.
…On Nov 1, 2017, 11:38 PM -0600, Ramya Rao ***@***.***>, wrote:
Sorry about that.
Yes, that's true. We should have a timeout on the process that is spawned to run gocode
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
I'll have to get a repro to understand whats happening behind the scenes |
Looks like this is not be the fault of VSCode or the extension. I found pqrs-org/Karabiner-Elements#545 and it appears to be the cause of my issue. The spike in load caused by |
Wow, I would have never guessed that |
No idea what is causing this or how to start debugging it, but here's what happens:
If I completely disable autocomplete:
The issue disappears. New issue though: now I don't have autocomplete. :)
The text was updated successfully, but these errors were encountered: