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

fix: High CPU Usage with Invalid Config #50

Merged

folke merged 1 commit into folke:main from danields761:main

Sep 30, 2023

Contributor

danields761 commented Sep 30, 2023 •

edited

Hello @folke,

I've created a pull request to fix a bug where Neovim chews up 100% of a CPU core when it encounters an invalid configuration.

The bug seems to be in edgy.util.with_retry, where it doesn't increase the retries variable, causing endless loops of retries and high CPU usage.

I've committed a fix. This should properly deal with invalid configurations and prevent Neovim from overloading the CPU.

Hope this helps out. Happy to chat more if needed!

Cheers,
Daniil


          fix: high CPU usage with invalid config (fixes #49)

b63f653

folke merged commit ee41d71 into folke:main

Owner

folke commented Sep 30, 2023

Thanks!

github-actions bot mentioned this pull request

chore(main): release 1.8.1 #51

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment