From: joaotavora@gmail.com (João Távora)
To: Vitalie Spinu <spinuvit@gmail.com>
Cc: Eli Zaretskii <eliz@gnu.org>,
dgutov@yandex.ru, monnier@IRO.UMontreal.CA, emacs-devel@gnu.org
Subject: Re: xref backends for elisp-related modes Was: Re: Bad moves with xref-find-definitions
Date: Tue, 28 Apr 2015 12:41:19 +0100 [thread overview]
Message-ID: <jjbmw1sqy3k.fsf@gmail.com> (raw)
In-Reply-To: <87383kcy0u.fsf@gmail.com> (Vitalie Spinu's message of "Tue, 28 Apr 2015 13:06:41 +0200")
Vitalie Spinu <spinuvit@gmail.com> writes:
> >>> Eli Zaretskii on Sun, 26 Apr 2015 17:51:18 +0300 wrote:
>
> > Beware: quite a few commands pop up *Help* buffers whose contents have
> > nothing to do with Lisp. Some examples:
>
> > . C-u C-x =
> > . C-h C-\
> > . M-x describe-current-coding-system RET
>
> > etc.
>
> It's because help-mode is not specialized enough. I think there should
> be `elisp-help-mode`, `describe-help-mode` etc. which are derived from
> `help-mode`.
Maybe yes. But anyway Eli's right, some help contexts have nothing to do
with elisp. I see two options:
1) I like this derived mode approach. To implement it perhaps we could
make `help-mode-setup' call some `help-mode-setup-function' so that
`with-help-window' does the right thing. Then we set this var in the
contexts we want. If this is the chosen approach, what should
`help-mode-setup-function' default to? Perhaps the mew derived
`elisp-help-mode' since that minimizes the number of describe-*'s to
touch.
2) Otherwise, we have to explicitly reset the backend on a per-context
basis. Now, should the default for help mode be `elisp-xref-find' and
we reset those three, or the other way around? Is my previous
(advice-add describe-thingy :after ...) strategy acceptable for this?
(BTW: the fix for apropos-mode and debugger-mode is not controversial
right?)
> Maybe obvious, but this struggle with the choice of the
> default backend wouldn't be there if xref was able to merge backends.
I don't see how. How would we remove elisp-xref-find as a backend in
certain help-mode contexts without touching the code where such
contexts are setup.
> (FWIW, I would also like to have elisp xref in info mode.)
But only for the elisp manual, right?
João
next prev parent reply other threads:[~2015-04-28 11:41 UTC|newest]
Thread overview: 63+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-23 15:07 Bad moves with xref-find-definitions Vitalie Spinu
2015-04-25 14:24 ` Stefan Monnier
2015-04-25 16:25 ` Dmitry Gutov
2015-04-25 17:42 ` Vitalie Spinu
2015-04-25 18:49 ` Vitalie Spinu
2015-04-25 19:07 ` Dmitry Gutov
2015-04-25 18:56 ` João Távora
2015-04-25 23:50 ` Dmitry Gutov
2015-04-26 11:51 ` xref backends for elisp-related modes Was: " João Távora
2015-04-26 13:56 ` Dmitry Gutov
2015-04-26 14:51 ` Eli Zaretskii
2015-04-28 11:06 ` Vitalie Spinu
2015-04-28 11:41 ` João Távora [this message]
2015-04-28 11:59 ` Vitalie Spinu
2015-04-28 15:17 ` Eli Zaretskii
2015-04-28 15:45 ` Vitalie Spinu
2015-04-28 16:01 ` Eli Zaretskii
2015-04-28 13:27 ` Stefan Monnier
2015-04-28 21:28 ` Dmitry Gutov
2015-04-29 12:35 ` Vitalie Spinu
2015-04-29 15:45 ` Eli Zaretskii
2015-04-28 15:15 ` Eli Zaretskii
2015-04-28 15:47 ` Vitalie Spinu
2015-04-28 16:03 ` Eli Zaretskii
2015-04-29 6:55 ` Helmut Eller
2015-04-29 12:40 ` Vitalie Spinu
2015-04-29 13:01 ` Helmut Eller
2015-04-29 15:30 ` Vitalie Spinu
2015-04-29 17:09 ` Dmitry Gutov
2015-04-29 12:47 ` Dmitry Gutov
2015-04-29 13:04 ` Helmut Eller
2015-04-29 13:12 ` Dmitry Gutov
2015-04-27 2:20 ` Stefan Monnier
2015-04-25 19:11 ` Dmitry Gutov
2015-04-25 20:43 ` Vitalie Spinu
2015-04-26 3:37 ` Dmitry Gutov
2015-04-26 6:38 ` Bozhidar Batsov
2015-04-26 18:41 ` Dmitry Gutov
2015-04-27 19:36 ` Bozhidar Batsov
2015-04-28 1:23 ` Dmitry Gutov
2015-04-28 11:30 ` Vitalie Spinu
2015-04-26 10:44 ` Vitalie Spinu
2015-04-26 13:14 ` Vitalie Spinu
2015-04-26 15:09 ` Dmitry Gutov
2015-04-26 16:23 ` Vitalie Spinu
2015-04-26 17:51 ` Dmitry Gutov
2015-04-26 20:56 ` Vitalie Spinu
2015-04-27 21:57 ` Dmitry Gutov
2015-04-26 3:34 ` Stefan Monnier
2015-04-26 11:31 ` Vitalie Spinu
2015-04-26 14:50 ` Eli Zaretskii
2015-04-26 15:12 ` Dmitry Gutov
2015-04-26 15:32 ` Eli Zaretskii
2015-04-26 15:20 ` Dmitry Gutov
2015-04-26 16:01 ` Vitalie Spinu
2015-04-26 17:26 ` Dmitry Gutov
2015-04-27 2:26 ` Stefan Monnier
2015-04-25 19:01 ` Dmitry Gutov
2015-04-25 20:34 ` Vitalie Spinu
2015-04-25 21:44 ` Vitalie Spinu
2015-04-26 0:20 ` Dmitry Gutov
2015-04-26 0:28 ` Dmitry Gutov
2015-04-28 15:31 ` Vitalie Spinu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=jjbmw1sqy3k.fsf@gmail.com \
--to=joaotavora@gmail.com \
--cc=dgutov@yandex.ru \
--cc=eliz@gnu.org \
--cc=emacs-devel@gnu.org \
--cc=monnier@IRO.UMontreal.CA \
--cc=spinuvit@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.