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

Add mode-class property where appropriate #448

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mssdvd
Copy link

@mssdvd mssdvd commented Oct 25, 2021

This PR adds the property mode-class with value special to
elfeed-search-mode and elfeed-show-mode.

According to the elisp docs every "special" mode should have this
property.

This commit adds the property `mode-class` with value `special` to
`elfeed-search-mode` and `elfeed-show-mode`.

According to the elisp docs every "special" mode should have this
property.
@mssdvd
Copy link
Author

mssdvd commented Nov 5, 2021

The motivation behind this PR is a performance issue that is triggered when Flycheck is enabled for elfeed-search buffers.

global-flycheck-mode enables Flycheck where it makes sense (e.g. not in the minibuffer, fundamental-mode etc.). For special buffers like elfeed-search the only way to let know Flycheck what to do is the property mode-class.

Without this patch Emacs freeze for many seconds, sometimes minutes.

@jiacai2050
Copy link

Following #317, With global-flycheck-mode on, my Emacs hang 2 minutes when update feeds. After disable it, Emacs only hang a few seconds.
Snip20220505_23

@skeeto Hope you can merge this.

@jiacai2050
Copy link

(setq flycheck-global-modes '(not . (elfeed-search-mode)))

For now, users can use this config to disable flycheck in elfeed-search-mode.

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

Successfully merging this pull request may close these issues.

2 participants