GNU bug report logs -
#19443
25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience
Previous Next
Reported by: Filipp Gunbin <fgunbin <at> fastmail.fm>
Date: Fri, 26 Dec 2014 17:07:02 UTC
Severity: wishlist
Found in version 25.0.50
Done: Filipp Gunbin <fgunbin <at> fastmail.fm>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 19443 in the body.
You can then email your comments to 19443 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#19443; Package
emacs.
(Fri, 26 Dec 2014 17:07:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Filipp Gunbin <fgunbin <at> fastmail.fm>:
New bug report received and forwarded. Copy sent to
bug-gnu-emacs <at> gnu.org.
(Fri, 26 Dec 2014 17:07:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
Here's the patch on the subject. I'll apply it if no one objects.
Filipp
diff --git a/etc/NEWS b/etc/NEWS
index 14933aa..9e0c9c7 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -338,6 +338,11 @@ The remainder were:
---
** `Info-fontify-maximum-menu-size' can be t for no limit.
++++
+** `info-display-manual' can now be given a prefix argument which (any
+non-nil value) directs the command to limit the choice of manual to
+currently visited manuals.
+
---
** ntlm.el has support for NTLM2.
diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index a2bee14..3119d80 100644
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2014-12-26 Filipp Gunbin <fgunbin <at> fastmail.fm>
+
+ * info.el (info-display-manual): Limit the choice of manuals to
+ currently visited if prefix argument is non-nil.
+
2014-12-25 Helmut Eller <eller.helmut <at> gmail.com>
Dmitry Gutov <dgutov <at> yandex.ru>
diff --git a/lisp/info.el b/lisp/info.el
index 7c4d7f3..c5e3b21 100644
--- a/lisp/info.el
+++ b/lisp/info.el
@@ -5277,13 +5277,15 @@ type returned by `Info-bookmark-make-record', which see."
(defun info-display-manual (manual)
"Display an Info buffer displaying MANUAL.
If there is an existing Info buffer for MANUAL, display it.
-Otherwise, visit the manual in a new Info buffer."
+Otherwise, visit the manual in a new Info buffer. In interactive
+use, a prefix argument directs this command to limit the choice
+to currently visited manuals."
(interactive
(list
(progn
(info-initialize)
(completing-read "Manual name: "
- (info--manual-names)
+ (info--manual-names current-prefix-arg)
nil t))))
(let ((blist (buffer-list))
(manual-re (concat "\\(/\\|\\`\\)" manual "\\(\\.\\|\\'\\)"))
@@ -5302,7 +5304,7 @@ Otherwise, visit the manual in a new Info buffer."
(info (Info-find-file manual)
(generate-new-buffer-name "*info*")))))
-(defun info--manual-names ()
+(defun info--manual-names (visited-only)
(let (names)
(dolist (buffer (buffer-list))
(with-current-buffer buffer
@@ -5313,11 +5315,12 @@ Otherwise, visit the manual in a new Info buffer."
(file-name-nondirectory Info-current-file))
names))))
(delete-dups (append (nreverse names)
- (all-completions
- ""
- (apply-partially 'Info-read-node-name-2
- Info-directory-list
- (mapcar 'car Info-suffix-list)))))))
+ (when (not visited-only)
+ (all-completions
+ ""
+ (apply-partially 'Info-read-node-name-2
+ Info-directory-list
+ (mapcar 'car Info-suffix-list))))))))
(provide 'info)
Information forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#19443; Package
emacs.
(Fri, 26 Dec 2014 20:38:01 GMT)
Full text and
rfc822 format available.
Message #8 received at 19443 <at> debbugs.gnu.org (full text, mbox):
Filipp Gunbin wrote:
> ++++
> +** `info-display-manual' can now be given a prefix argument which (any
> +non-nil value) directs the command to limit the choice of manual to
> +currently visited manuals.
"+++" is inappropriate since your patch includes no doc/ changes.
Information forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#19443; Package
emacs.
(Fri, 26 Dec 2014 21:56:02 GMT)
Full text and
rfc822 format available.
Message #11 received at 19443 <at> debbugs.gnu.org (full text, mbox):
> From: Filipp Gunbin <fgunbin <at> fastmail.fm>
> Date: Fri, 26 Dec 2014 20:06:45 +0300
>
> Here's the patch on the subject. I'll apply it if no one objects.
Fine with me, thanks.
One minor comment about the doc string: you are talking about "the
choice", but no choice was mentioned before in the doc string. This
might confuse the user. I think you should mention completion in
order to make this clear.
Information forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#19443; Package
emacs.
(Sat, 27 Dec 2014 00:01:02 GMT)
Full text and
rfc822 format available.
Message #14 received at 19443 <at> debbugs.gnu.org (full text, mbox):
On 26/12/2014 15:37 -0500, Glenn Morris wrote:
> Filipp Gunbin wrote:
>
>> ++++
>> +** `info-display-manual' can now be given a prefix argument which (any
>> +non-nil value) directs the command to limit the choice of manual to
>> +currently visited manuals.
>
> "+++" is inappropriate since your patch includes no doc/ changes.
But etc/CONTRIBUTE says:
"Think about whether your change requires updating the documentation
(both manuals and doc-strings). If you know it does not, mark the NEWS
entry with "---". If you know that *all* the necessary documentation
updates have been made, mark the entry with "+++". Otherwise do not mark
it."
And I've updated the doc-string.
Am I missing something?
Information forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#19443; Package
emacs.
(Sat, 27 Dec 2014 00:10:02 GMT)
Full text and
rfc822 format available.
Message #17 received at 19443 <at> debbugs.gnu.org (full text, mbox):
On 26/12/2014 23:55 +0200, Eli Zaretskii wrote:
>> From: Filipp Gunbin <fgunbin <at> fastmail.fm>
>> Date: Fri, 26 Dec 2014 20:06:45 +0300
>>
>> Here's the patch on the subject. I'll apply it if no one objects.
>
> Fine with me, thanks.
>
> One minor comment about the doc string: you are talking about "the
> choice", but no choice was mentioned before in the doc string. This
> might confuse the user. I think you should mention completion in
> order to make this clear.
Ok, changed to "completion alternatives". Thanks.
diff --git a/etc/NEWS b/etc/NEWS
index 14933aa..315332f 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -338,6 +338,11 @@ The remainder were:
---
** `Info-fontify-maximum-menu-size' can be t for no limit.
++++
+** `info-display-manual' can now be given a prefix argument which (any
+non-nil value) directs the command to limit the completion
+alternatives to currently visited manuals.
+
---
** ntlm.el has support for NTLM2.
diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index a2bee14..d3d587d 100644
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2014-12-26 Filipp Gunbin <fgunbin <at> fastmail.fm>
+
+ * info.el (info-display-manual): Limit the completion alternatives
+ to currently visited manuals if prefix argument is non-nil.
+
2014-12-25 Helmut Eller <eller.helmut <at> gmail.com>
Dmitry Gutov <dgutov <at> yandex.ru>
diff --git a/lisp/info.el b/lisp/info.el
index 7c4d7f3..33e982d 100644
--- a/lisp/info.el
+++ b/lisp/info.el
@@ -5277,13 +5277,15 @@ type returned by `Info-bookmark-make-record', which see."
(defun info-display-manual (manual)
"Display an Info buffer displaying MANUAL.
If there is an existing Info buffer for MANUAL, display it.
-Otherwise, visit the manual in a new Info buffer."
+Otherwise, visit the manual in a new Info buffer. In interactive
+use, a prefix argument directs this command to limit the
+completion alternatives to currently visited manuals."
(interactive
(list
(progn
(info-initialize)
(completing-read "Manual name: "
- (info--manual-names)
+ (info--manual-names current-prefix-arg)
nil t))))
(let ((blist (buffer-list))
(manual-re (concat "\\(/\\|\\`\\)" manual "\\(\\.\\|\\'\\)"))
@@ -5302,7 +5304,7 @@ Otherwise, visit the manual in a new Info buffer."
(info (Info-find-file manual)
(generate-new-buffer-name "*info*")))))
-(defun info--manual-names ()
+(defun info--manual-names (visited-only)
(let (names)
(dolist (buffer (buffer-list))
(with-current-buffer buffer
@@ -5313,11 +5315,12 @@ Otherwise, visit the manual in a new Info buffer."
(file-name-nondirectory Info-current-file))
names))))
(delete-dups (append (nreverse names)
- (all-completions
- ""
- (apply-partially 'Info-read-node-name-2
- Info-directory-list
- (mapcar 'car Info-suffix-list)))))))
+ (when (not visited-only)
+ (all-completions
+ ""
+ (apply-partially 'Info-read-node-name-2
+ Info-directory-list
+ (mapcar 'car Info-suffix-list))))))))
(provide 'info)
Information forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#19443; Package
emacs.
(Sat, 27 Dec 2014 07:34:02 GMT)
Full text and
rfc822 format available.
Message #20 received at 19443 <at> debbugs.gnu.org (full text, mbox):
> From: Filipp Gunbin <fgunbin <at> fastmail.fm>
> Date: Sat, 27 Dec 2014 03:00:23 +0300
> Cc: 19443 <at> debbugs.gnu.org
>
> On 26/12/2014 15:37 -0500, Glenn Morris wrote:
>
> > Filipp Gunbin wrote:
> >
> >> ++++
> >> +** `info-display-manual' can now be given a prefix argument which (any
> >> +non-nil value) directs the command to limit the choice of manual to
> >> +currently visited manuals.
> >
> > "+++" is inappropriate since your patch includes no doc/ changes.
>
> But etc/CONTRIBUTE says:
>
> "Think about whether your change requires updating the documentation
> (both manuals and doc-strings). If you know it does not, mark the NEWS
> entry with "---". If you know that *all* the necessary documentation
> updates have been made, mark the entry with "+++". Otherwise do not mark
> it."
>
> And I've updated the doc-string.
>
> Am I missing something?
Yes, you are missing info.texi, which does describe this command. I
think a sentence about the significance of the prefix argument in this
case would be appropriate there. If you do that, then +++ will be
appropriate.
Thanks.
(In general, "C-h k" AND "C-h f" are useful to see if there's any
documentation in any manual for a key or a function. There's no such
command for variables, unfortunately.)
Information forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#19443; Package
emacs.
(Sat, 27 Dec 2014 15:17:01 GMT)
Full text and
rfc822 format available.
Message #23 received at 19443 <at> debbugs.gnu.org (full text, mbox):
> "Think about whether your change requires updating the documentation
> (both manuals and doc-strings). If you know it does not, mark the NEWS
> entry with "---". If you know that *all* the necessary documentation
> updates have been made, mark the entry with "+++". Otherwise do not mark
> it."
The --- and +++ in etc/NEWS only relate to the manual.
The docstrings should *always* be updated right away.
Stefan
Information forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#19443; Package
emacs.
(Sat, 27 Dec 2014 16:02:02 GMT)
Full text and
rfc822 format available.
Message #26 received at 19443 <at> debbugs.gnu.org (full text, mbox):
> (In general, "C-h k" AND "C-h f" are useful to see if there's any
> documentation in any manual for a key or a function.
Hmm... I must be missing something, because neither of those gives me
any indication about the presence of related info in the manual,
Stefan
Information forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#19443; Package
emacs.
(Sat, 27 Dec 2014 18:35:02 GMT)
Full text and
rfc822 format available.
Message #29 received at 19443 <at> debbugs.gnu.org (full text, mbox):
> From: Stefan Monnier <monnier <at> iro.umontreal.ca>
> Cc: Filipp Gunbin <fgunbin <at> fastmail.fm>, 19443 <at> debbugs.gnu.org
> Date: Sat, 27 Dec 2014 11:01:17 -0500
>
> > (In general, "C-h k" AND "C-h f" are useful to see if there's any
> > documentation in any manual for a key or a function.
>
> Hmm... I must be missing something, because neither of those gives me
> any indication about the presence of related info in the manual,
Strange, I'm positive I typed "C-h K" and "C-h F".
Information forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#19443; Package
emacs.
(Tue, 30 Dec 2014 11:47:01 GMT)
Full text and
rfc822 format available.
Message #32 received at 19443 <at> debbugs.gnu.org (full text, mbox):
On 27/12/2014 10:16 -0500, Stefan Monnier wrote:
>> "Think about whether your change requires updating the documentation
>> (both manuals and doc-strings). If you know it does not, mark the NEWS
>> entry with "---". If you know that *all* the necessary documentation
>> updates have been made, mark the entry with "+++". Otherwise do not mark
>> it."
>
> The --- and +++ in etc/NEWS only relate to the manual.
> The docstrings should *always* be updated right away.
>
>
> Stefan
Thanks Stefan and Eli,
Here's the revised patch. I also updated the CONTRIBUTE file in
accordance with what you wrote. Could you please review?
Filipp
diff --git a/CONTRIBUTE b/CONTRIBUTE
index 0e019d3..5cf015f 100644
--- a/CONTRIBUTE
+++ b/CONTRIBUTE
@@ -180,10 +180,12 @@ by following links from http://savannah.gnu.org/mail/?group=emacs .
Any change that matters to end-users should have an entry in etc/NEWS.
-Think about whether your change requires updating the documentation
-(both manuals and doc-strings). If you know it does not, mark the NEWS
-entry with "---". If you know that *all* the necessary documentation
-updates have been made, mark the entry with "+++". Otherwise do not mark it.
+Doc-strings should be updated together with the code.
+
+Think about whether your change requires updating the manuals. If you
+know it does not, mark the NEWS entry with "---". If you know
+that *all* the necessary documentation updates have been made, mark
+the entry with "+++". Otherwise do not mark it.
** Understanding Emacs Internals.
diff --git a/doc/misc/info.texi b/doc/misc/info.texi
index a3a14a3..0e2e64f 100644
--- a/doc/misc/info.texi
+++ b/doc/misc/info.texi
@@ -1151,7 +1151,10 @@ switches to the buffer @file{*info*<2>}, creating it if necessary.
If you have created many Info buffers in Emacs, you might find it
difficult to remember which buffer is showing which manual. You can
use the command @kbd{M-x info-display-manual} to show an Info manual
-by name, reusing an existing buffer if there is one.
+by name, reusing an existing buffer if there is one. When given a
+prefix argument, this command limits the completion alternatives to
+currently visited info files, thus giving a convenient way to switch
+between several manuals.
@node Emacs Info Variables
@section Emacs Info-mode Variables
diff --git a/etc/NEWS b/etc/NEWS
index ae0cb70..ec5fe0d 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -338,6 +338,11 @@ The remainder were:
---
** `Info-fontify-maximum-menu-size' can be t for no limit.
++++
+** `info-display-manual' can now be given a prefix argument which (any
+non-nil value) directs the command to limit the completion
+alternatives to currently visited manuals.
+
---
** ntlm.el has support for NTLM2.
diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 6249a30..86ed70f 100644
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2014-12-30 Filipp Gunbin <fgunbin <at> fastmail.fm>
+
+ * info.el (info-display-manual): Limit the completion alternatives
+ to currently visited manuals if prefix argument is non-nil.
+
2014-12-30 Dmitry Gutov <dgutov <at> yandex.ru>
* menu-bar.el (menu-bar-goto-uses-etags-p): New function.
diff --git a/lisp/info.el b/lisp/info.el
index 7c4d7f3..33e982d 100644
--- a/lisp/info.el
+++ b/lisp/info.el
@@ -5277,13 +5277,15 @@ type returned by `Info-bookmark-make-record', which see."
(defun info-display-manual (manual)
"Display an Info buffer displaying MANUAL.
If there is an existing Info buffer for MANUAL, display it.
-Otherwise, visit the manual in a new Info buffer."
+Otherwise, visit the manual in a new Info buffer. In interactive
+use, a prefix argument directs this command to limit the
+completion alternatives to currently visited manuals."
(interactive
(list
(progn
(info-initialize)
(completing-read "Manual name: "
- (info--manual-names)
+ (info--manual-names current-prefix-arg)
nil t))))
(let ((blist (buffer-list))
(manual-re (concat "\\(/\\|\\`\\)" manual "\\(\\.\\|\\'\\)"))
@@ -5302,7 +5304,7 @@ Otherwise, visit the manual in a new Info buffer."
(info (Info-find-file manual)
(generate-new-buffer-name "*info*")))))
-(defun info--manual-names ()
+(defun info--manual-names (visited-only)
(let (names)
(dolist (buffer (buffer-list))
(with-current-buffer buffer
@@ -5313,11 +5315,12 @@ Otherwise, visit the manual in a new Info buffer."
(file-name-nondirectory Info-current-file))
names))))
(delete-dups (append (nreverse names)
- (all-completions
- ""
- (apply-partially 'Info-read-node-name-2
- Info-directory-list
- (mapcar 'car Info-suffix-list)))))))
+ (when (not visited-only)
+ (all-completions
+ ""
+ (apply-partially 'Info-read-node-name-2
+ Info-directory-list
+ (mapcar 'car Info-suffix-list))))))))
(provide 'info)
Information forwarded
to
bug-gnu-emacs <at> gnu.org:
bug#19443; Package
emacs.
(Tue, 30 Dec 2014 16:11:01 GMT)
Full text and
rfc822 format available.
Message #35 received at 19443 <at> debbugs.gnu.org (full text, mbox):
> From: Filipp Gunbin <fgunbin <at> fastmail.fm>
> Cc: Glenn Morris <rgm <at> gnu.org>, 19443 <at> debbugs.gnu.org
> Date: Tue, 30 Dec 2014 14:46:30 +0300
>
> Thanks Stefan and Eli,
>
> Here's the revised patch. I also updated the CONTRIBUTE file in
> accordance with what you wrote. Could you please review?
LGTM, thanks.
Reply sent
to
Filipp Gunbin <fgunbin <at> fastmail.fm>:
You have taken responsibility.
(Wed, 31 Dec 2014 15:45:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
Filipp Gunbin <fgunbin <at> fastmail.fm>:
bug acknowledged by developer.
(Wed, 31 Dec 2014 15:45:03 GMT)
Full text and
rfc822 format available.
Message #40 received at 19443-done <at> debbugs.gnu.org (full text, mbox):
On 30/12/2014 18:10 +0200, Eli Zaretskii wrote:
>> From: Filipp Gunbin <fgunbin <at> fastmail.fm>
>> Cc: Glenn Morris <rgm <at> gnu.org>, 19443 <at> debbugs.gnu.org
>> Date: Tue, 30 Dec 2014 14:46:30 +0300
>>
>> Thanks Stefan and Eli,
>>
>> Here's the revised patch. I also updated the CONTRIBUTE file in
>> accordance with what you wrote. Could you please review?
>
> LGTM, thanks.
Applied.
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org.
(Thu, 29 Jan 2015 12:24:04 GMT)
Full text and
rfc822 format available.
This bug report was last modified 10 years and 203 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.