]> git.eshelyaron.com Git - emacs.git/commit
Eglot: respect user's Eldoc configuration by default
authorJoão Távora <joaotavora@gmail.com>
Mon, 20 Feb 2023 22:43:50 +0000 (22:43 +0000)
committerJoão Távora <joaotavora@gmail.com>
Mon, 20 Feb 2023 22:44:30 +0000 (22:44 +0000)
commite83c78b8c7784254c2c6f043530ab325c2fa7f16
tree33b64aa3212d88b42ae276152999ed300c741ab3
parent5d0b45cd67b383bc777dcf7589517e51e58f5a21
Eglot: respect user's Eldoc configuration by default

This change addresses the problems reported in many Elglot reports
dating back to early 2021 at least:

  https://github.com/joaotavora/eglot/issues/648
  https://github.com/joaotavora/eglot/issues/894
  https://github.com/joaotavora/eglot/issues/920
  https://github.com/joaotavora/eglot/issues/1031
  https://github.com/joaotavora/eglot/issues/1171

In one form or another, the reports point out that the multiple pieces
of information about the "thing at point" made available by the LSP
server are not all being considered by the ElDoc system.

The reason for this is Eglot setting/trampling the variables
'eldoc-documentation-strategy' and 'eldoc-documentation-functions' in
its minor more entry function.

The reason it did that is historical and is partially described in the
issues above.  But, evidently, it never made much sense, because so
many people want to override it, which requires setting
'eldoc-documentation-strategy' to the non-default value
'eldoc-documentation-compose'.

The problem was made worse by the fact that setting it as usual in
either the Customize menu or their init file didn't work, requiring a
fairly complex Elisp snippet.  That is now solved as of this commit.

If the user does not do any setting, then Eglot works basically the
same as before (i.e. shows only one piece of information).

It is arguable that the default value for
'eldoc-documentation-strategy' should change globally to
'eldoc-documentation-compose', but that has other subtle implications
and is not part of this commit.

* lisp/progmodes/eglot.el (eglot--managed-mode): Don't set Eldoc
variables greedily.
lisp/progmodes/eglot.el