Skip to content

Add mode-class property where appropriate#448

Open
mssdvd wants to merge 1 commit intoskeeto:masterfrom
mssdvd:mode-class-prop
Open

Add mode-class property where appropriate#448
mssdvd wants to merge 1 commit intoskeeto:masterfrom
mssdvd:mode-class-prop

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