Skip to content

Conversation

nelhage
Copy link
Contributor

@nelhage nelhage commented Aug 19, 2025

Emacs HEAD has obsoleted revert-buffer-in-progress-p: emacs-mirror/emacs@1fc4bb1

We can't use the new name because it doesn't exist on older emacs versions, so just ignore the warning for now.

Emacs HEAD has obsoleted `revert-buffer-in-progress-p`:
https://github.com/emacs-mirror/emacs/commit/
1fc4bb1fead205e425f69fa803a71f4e446d48a8

We can't use the new name because it doesn't exist on older emacs
versions, so just ignore the warning for now.
@nelhage
Copy link
Contributor Author

nelhage commented Aug 20, 2025

I don't love the random warning suppressions in two different places. The best other option I came up with was sprinkling with-suppressed-warnings around the defuns, which I also don't love.

Maybe conditionally doing a defvaralias of an lsp--revert-buffer-in-progress-p to either the old or the new name would be cleaner. Any opinions?

And now we no longer need to suppress the warning
@nelhage
Copy link
Contributor Author

nelhage commented Aug 20, 2025

Okay, I like that better after all.

@jcs090218
Copy link
Member

The link doesn't seem to point to the relevant commit. 🤔 https://github.com/emacs-mirror/emacs/commit/

@nelhage
Copy link
Contributor Author

nelhage commented Aug 21, 2025

The link doesn't seem to point to the relevant commit.

Whoops, the link got line-broken somehow. Should work now -- emacs-mirror/emacs@1fc4bb1

@jcs090218 jcs090218 merged commit 19ba542 into emacs-lsp:master Aug 22, 2025
33 of 34 checks passed
@jcs090218
Copy link
Member

Thank you! :)

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