unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Chong Yidong <cyd@gnu.org>
Cc: 10771@debbugs.gnu.org
Subject: bug#10771: TAB completion for info-display-manual
Date: Sun, 02 Dec 2012 19:25:05 +0200	[thread overview]
Message-ID: <8338zotlhq.fsf@gnu.org> (raw)
In-Reply-To: <87a9txlzqa.fsf@gnu.org>

> From: Chong Yidong <cyd@gnu.org>
> Cc: Juri Linkov <juri@jurta.org>,  10771@debbugs.gnu.org,  rgm@gnu.org
> Date: Sun, 02 Dec 2012 14:44:45 +0800
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> > Thanks, but this is wrong: it should offer completion only on Info
> > buffers that already exist in the current session.  Completion on
> > manuals installed on the system is perhaps for "C-u C-h i" or some
> > variation of it.
> >
> > The purpose of this command is to assist with switching between
> > manuals that are already loaded, not in adding others.
> 
> I disagree; completion should be provided for any value that the command
> accepts as an argument, so Juri's patch does the right thing.

No, it doesn't.  The command looks up its argument among Info
_buffers_ first, and only if not found, falls back to searching the
manuals installed on the system.  So at the very least, the completion
should include in the collection the manuals loaded into Info buffers
in the current session.  Because manuals loaded into my Info buffers
do not necessarily reflect Info files on my disk.

> If one wishes to get fancy, I suppose one could define a completion
> category override to cause completion to favor manuals in existing Info
> buffers.

There's no need to favor existing buffers, but there is a need to
include them in the completion.





  reply	other threads:[~2012-12-02 17:25 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-09  6:36 bug#10771: TAB completion for info-display-manual Glenn Morris
2012-02-09  9:28 ` Juri Linkov
2012-02-09 18:42   ` Eli Zaretskii
2012-02-10 10:17     ` Juri Linkov
2012-12-02  6:44     ` Chong Yidong
2012-12-02 17:25       ` Eli Zaretskii [this message]
2012-12-02 21:12         ` Juri Linkov
2012-12-02 21:32           ` Eli Zaretskii
2012-12-02 21:57             ` Juri Linkov
2012-12-03  3:37               ` Eli Zaretskii
2012-12-03  6:57                 ` Chong Yidong
2012-12-03 16:31                   ` Eli Zaretskii

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

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8338zotlhq.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=10771@debbugs.gnu.org \
    --cc=cyd@gnu.org \
    /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 public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).