* bug#19443: 25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience @ 2014-12-26 17:06 Filipp Gunbin 2014-12-26 20:37 ` Glenn Morris 2014-12-26 21:55 ` Eli Zaretskii 0 siblings, 2 replies; 12+ messages in thread From: Filipp Gunbin @ 2014-12-26 17:06 UTC (permalink / raw) To: 19443 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@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@gmail.com> Dmitry Gutov <dgutov@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) ^ permalink raw reply related [flat|nested] 12+ messages in thread
* bug#19443: 25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience 2014-12-26 17:06 bug#19443: 25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience Filipp Gunbin @ 2014-12-26 20:37 ` Glenn Morris 2014-12-27 0:00 ` Filipp Gunbin 2014-12-26 21:55 ` Eli Zaretskii 1 sibling, 1 reply; 12+ messages in thread From: Glenn Morris @ 2014-12-26 20:37 UTC (permalink / raw) To: Filipp Gunbin; +Cc: 19443 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. ^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#19443: 25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience 2014-12-26 20:37 ` Glenn Morris @ 2014-12-27 0:00 ` Filipp Gunbin 2014-12-27 7:33 ` Eli Zaretskii 2014-12-27 15:16 ` Stefan Monnier 0 siblings, 2 replies; 12+ messages in thread From: Filipp Gunbin @ 2014-12-27 0:00 UTC (permalink / raw) To: Glenn Morris; +Cc: 19443 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? ^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#19443: 25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience 2014-12-27 0:00 ` Filipp Gunbin @ 2014-12-27 7:33 ` Eli Zaretskii 2014-12-27 16:01 ` Stefan Monnier 2014-12-27 15:16 ` Stefan Monnier 1 sibling, 1 reply; 12+ messages in thread From: Eli Zaretskii @ 2014-12-27 7:33 UTC (permalink / raw) To: Filipp Gunbin; +Cc: 19443 > From: Filipp Gunbin <fgunbin@fastmail.fm> > Date: Sat, 27 Dec 2014 03:00:23 +0300 > Cc: 19443@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.) ^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#19443: 25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience 2014-12-27 7:33 ` Eli Zaretskii @ 2014-12-27 16:01 ` Stefan Monnier 2014-12-27 18:34 ` Eli Zaretskii 0 siblings, 1 reply; 12+ messages in thread From: Stefan Monnier @ 2014-12-27 16:01 UTC (permalink / raw) To: Eli Zaretskii; +Cc: 19443, Filipp Gunbin > (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 ^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#19443: 25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience 2014-12-27 16:01 ` Stefan Monnier @ 2014-12-27 18:34 ` Eli Zaretskii 0 siblings, 0 replies; 12+ messages in thread From: Eli Zaretskii @ 2014-12-27 18:34 UTC (permalink / raw) To: Stefan Monnier; +Cc: 19443, fgunbin > From: Stefan Monnier <monnier@iro.umontreal.ca> > Cc: Filipp Gunbin <fgunbin@fastmail.fm>, 19443@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". ^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#19443: 25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience 2014-12-27 0:00 ` Filipp Gunbin 2014-12-27 7:33 ` Eli Zaretskii @ 2014-12-27 15:16 ` Stefan Monnier 2014-12-30 11:46 ` Filipp Gunbin 1 sibling, 1 reply; 12+ messages in thread From: Stefan Monnier @ 2014-12-27 15:16 UTC (permalink / raw) To: Filipp Gunbin; +Cc: 19443 > "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 ^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#19443: 25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience 2014-12-27 15:16 ` Stefan Monnier @ 2014-12-30 11:46 ` Filipp Gunbin 2014-12-30 16:10 ` Eli Zaretskii 0 siblings, 1 reply; 12+ messages in thread From: Filipp Gunbin @ 2014-12-30 11:46 UTC (permalink / raw) To: Stefan Monnier, Eli Zaretskii; +Cc: 19443 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@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@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) ^ permalink raw reply related [flat|nested] 12+ messages in thread
* bug#19443: 25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience 2014-12-30 11:46 ` Filipp Gunbin @ 2014-12-30 16:10 ` Eli Zaretskii 2014-12-31 15:44 ` Filipp Gunbin 0 siblings, 1 reply; 12+ messages in thread From: Eli Zaretskii @ 2014-12-30 16:10 UTC (permalink / raw) To: Filipp Gunbin; +Cc: 19443 > From: Filipp Gunbin <fgunbin@fastmail.fm> > Cc: Glenn Morris <rgm@gnu.org>, 19443@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. ^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#19443: 25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience 2014-12-30 16:10 ` Eli Zaretskii @ 2014-12-31 15:44 ` Filipp Gunbin 0 siblings, 0 replies; 12+ messages in thread From: Filipp Gunbin @ 2014-12-31 15:44 UTC (permalink / raw) To: Eli Zaretskii; +Cc: 19443-done On 30/12/2014 18:10 +0200, Eli Zaretskii wrote: >> From: Filipp Gunbin <fgunbin@fastmail.fm> >> Cc: Glenn Morris <rgm@gnu.org>, 19443@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. ^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#19443: 25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience 2014-12-26 17:06 bug#19443: 25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience Filipp Gunbin 2014-12-26 20:37 ` Glenn Morris @ 2014-12-26 21:55 ` Eli Zaretskii 2014-12-27 0:08 ` Filipp Gunbin 1 sibling, 1 reply; 12+ messages in thread From: Eli Zaretskii @ 2014-12-26 21:55 UTC (permalink / raw) To: Filipp Gunbin; +Cc: 19443 > From: Filipp Gunbin <fgunbin@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. ^ permalink raw reply [flat|nested] 12+ messages in thread
* bug#19443: 25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience 2014-12-26 21:55 ` Eli Zaretskii @ 2014-12-27 0:08 ` Filipp Gunbin 0 siblings, 0 replies; 12+ messages in thread From: Filipp Gunbin @ 2014-12-27 0:08 UTC (permalink / raw) To: Eli Zaretskii; +Cc: 19443 On 26/12/2014 23:55 +0200, Eli Zaretskii wrote: >> From: Filipp Gunbin <fgunbin@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@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@gmail.com> Dmitry Gutov <dgutov@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) ^ permalink raw reply related [flat|nested] 12+ messages in thread
end of thread, other threads:[~2014-12-31 15:44 UTC | newest] Thread overview: 12+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-12-26 17:06 bug#19443: 25.0.50; `info-display-manual' could limit the choice to currently visited manuals if given a prefix arg for convenience Filipp Gunbin 2014-12-26 20:37 ` Glenn Morris 2014-12-27 0:00 ` Filipp Gunbin 2014-12-27 7:33 ` Eli Zaretskii 2014-12-27 16:01 ` Stefan Monnier 2014-12-27 18:34 ` Eli Zaretskii 2014-12-27 15:16 ` Stefan Monnier 2014-12-30 11:46 ` Filipp Gunbin 2014-12-30 16:10 ` Eli Zaretskii 2014-12-31 15:44 ` Filipp Gunbin 2014-12-26 21:55 ` Eli Zaretskii 2014-12-27 0:08 ` Filipp Gunbin
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).