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

async_switch_im = true will cause Neovim to crash on Windows #17

Closed
xarthurx opened this issue Sep 8, 2023 · 7 comments
Closed

async_switch_im = true will cause Neovim to crash on Windows #17

xarthurx opened this issue Sep 8, 2023 · 7 comments

Comments

@xarthurx
Copy link
Contributor

xarthurx commented Sep 8, 2023

async_switch_im = false all fine.

But once turned on to true , Neovim will either crash or halt for a long time, while the cmd window keep popping up and shutdown.
@keaising
Copy link
Owner

keaising commented Sep 8, 2023 •
edited
Loading

This is the reason that I provide an option for you to turn off async.

Neovim on Windows has a lot of wired behaviors, I think I can only provide a it works solution.

If you can provide a better solution to fix this issue, PR is welcome.

@xarthurx
Copy link
Contributor Author

xarthurx commented Sep 8, 2023

See. I actually don't know much about the mechanism behind nvim...

Better to mention it in the README.md?
Spent quite some time to debug the problem.

@keaising
Copy link
Owner

keaising commented Sep 9, 2023

To be honest, there are at least three users complained about this issue, but I don't even know actually reason and what senarios you meet.
For someone async is ok, for another one sync is ok, but I can't reproduce none of your issues, and I don't know how to descripe the issue can help you to fix this issue.

@xarthurx
Copy link
Contributor Author

Probably just a comment pointing to async param is enough.
Or leave this issue open.
But anyway, it depends on you.

@keaising
Copy link
Owner

Could you please give me an example about a comment pointing to async param is enough. ?

@xarthurx xarthurx mentioned this issue Sep 11, 2023
Merged
@xarthurx
Copy link
Contributor Author

Could you please give me an example about a comment pointing to async param is enough. ?

I submitted a PR for the Readme. Please feel free to modify it.

@keaising
Copy link
Owner

Looks good, merged.

Thank you!

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

No branches or pull requests
2 participants
@xarthurx @keaising