GNU bug report logs - #10771
TAB completion for info-display-manual

Previous Next

Full log

Message #28 received at 10771 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Juri Linkov <juri <at> jurta.org>
Cc: 10771 <at> debbugs.gnu.org, cyd <at> gnu.org, rgm <at> gnu.org
Subject: Re: bug#10771: TAB completion for info-display-manual
Date: Sun, 02 Dec 2012 23:32:28 +0200

> From: Juri Linkov <juri <at> jurta.org>
> Cc: Chong Yidong <cyd <at> gnu.org>,  10771 <at> debbugs.gnu.org,  rgm <at> gnu.org
> Date: Sun, 02 Dec 2012 23:12:35 +0200
> 
> > Because manuals loaded into my Info buffers do not necessarily reflect
> > Info files on my disk.
> 
> Is this because you delete Info files from the disk leaving open
> their Info buffers?

No, it's because I don't want to be restricted to manuals found in
directories that are mentioned in Info-directory-list.

In general, the Info buffers in the session should not be restricted
to what "M-x info" can find, because we have "C-u C-h i" and other
methods of loading manuals by their exact file name.

> I'm not sure whether additional complexity is necessary to handle
> completion for such rare cases

Well, _I_ wrote this command, and I needed it precisely because I have
dozens of Info manuals loaded in my usual Emacs session.  This case is
not rare for me.


> but at least
> the minibuffer should allow you to enter such manual names.
> I mean changing the REQUIRE-MATCH arg of `completing-read' from t to nil,
> so you can still enter these manual names like at the time when
> this command accepted a plain string before the latest change.
> 
> === modified file 'lisp/info.el'
> --- lisp/info.el	2012-12-02 20:07:04 +0000
> +++ lisp/info.el	2012-12-02 21:11:17 +0000
> @@ -5138,7 +5138,7 @@ (defun info-display-manual (manual)
>  		       (apply-partially 'Info-read-node-name-2
>  					Info-directory-list
>  					(mapcar 'car Info-suffix-list))
> -		       nil t))))
> +		       nil nil))))
>    (let ((blist (buffer-list))
>  	(manual-re (concat "\\(/\\|\\`\\)" manual "\\(\\.\\|\\'\\)"))
>  	(case-fold-search t)

How is this better than the original command against which this bug
was filed?

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.