diff options
author | João Távora <joaotavora@gmail.com> | 2022-09-08 11:53:11 +0100 |
---|---|---|
committer | João Távora <joaotavora@gmail.com> | 2022-09-09 00:37:38 +0100 |
commit | 41a42e631bd798151130097feafa6f535161b9de (patch) | |
tree | 87f6dcdf57a582dd888cecd8e325030cb1957523 /test/lisp/emacs-lisp/bytecomp-resources/error-lexical-var-with-add-hook.el | |
parent | e5b021c01fceea02b7e6622cde0a347b842ca6f3 (diff) | |
download | emacs-41a42e631bd798151130097feafa6f535161b9de.tar.gz emacs-41a42e631bd798151130097feafa6f535161b9de.tar.bz2 emacs-41a42e631bd798151130097feafa6f535161b9de.zip |
Don't return poorly supported "special elements" in eglot-imenu
Fix https://github.com/joaotavora/eglot/issues/758, https://github.com/joaotavora/eglot/issues/536, https://github.com/joaotavora/eglot/issues/535.
Eglot's eglot-imenu returned a structure compliant with the rules
outlined in imenu--index-alist. In particular, it returned some
elements of the form
(INDEX-NAME POSITION GOTO-FN ARGUMENTS...)
The original intention (mine) must have been to allow fancy
highlighting of the position navigated to with a custom GOTO-FN.
Not only was access to that fanciness never implemented, but many
other imenu frontends do not support such elements.
See for example https://github.com/joaotavora/eglot/issues/758, https://github.com/joaotavora/eglot/issues/536, https://github.com/joaotavora/eglot/issues/535. And also related issues in other
packages:
https://github.com/IvanMalison/flimenu/issues/6
https://github.com/bmag/imenu-list/issues/58
So it's best to remove this problematic feature for now. It can be
added back later.
* eglot.el (eglot-imenu): Simplify.
* NEWS.md: Mention change
Diffstat (limited to 'test/lisp/emacs-lisp/bytecomp-resources/error-lexical-var-with-add-hook.el')
0 files changed, 0 insertions, 0 deletions