* bug#12436: 24.2.50; rename `(next|previous)-multiframe-window' @ 2012-09-13 16:30 Drew Adams 2016-04-28 17:23 ` Lars Ingebrigtsen 0 siblings, 1 reply; 8+ messages in thread From: Drew Adams @ 2012-09-13 16:30 UTC (permalink / raw) To: 12436 Rename these commands and deprecate the old names. There is no such thing as a multiframe window, which would mean a window on more than one frame. These commands should be called something like: `(next|previous)-window-any-frame'. In GNU Emacs 24.2.50.1 (i386-mingw-nt5.1.2600) of 2012-09-02 on MARVIN Bzr revision: 109861 eggert@cs.ucla.edu-20120902171035-7mzihil3xd6bjfiy Windowing system distributor `Microsoft Corp.', version 5.1.2600 Configured using: `configure --with-gcc (4.6) --no-opt --enable-checking --cflags -ID:/devel/emacs/libs/libXpm-3.5.8/include -ID:/devel/emacs/libs/libXpm-3.5.8/src -ID:/devel/emacs/libs/libpng-dev_1.4.3-1/include -ID:/devel/emacs/libs/zlib-dev_1.2.5-2/include -ID:/devel/emacs/libs/giflib-4.1.4-1/include -ID:/devel/emacs/libs/jpeg-6b-4/include -ID:/devel/emacs/libs/tiff-3.8.2-1/include -ID:/devel/emacs/libs/gnutls-3.0.9/include -ID:/devel/emacs/libs/libiconv-1.13.1-1-dev/include -ID:/devel/emacs/libs/libxml2-2.7.8/include/libxml2' ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#12436: 24.2.50; rename `(next|previous)-multiframe-window' 2012-09-13 16:30 bug#12436: 24.2.50; rename `(next|previous)-multiframe-window' Drew Adams @ 2016-04-28 17:23 ` Lars Ingebrigtsen 2019-10-30 22:30 ` Stefan Kangas 0 siblings, 1 reply; 8+ messages in thread From: Lars Ingebrigtsen @ 2016-04-28 17:23 UTC (permalink / raw) To: Drew Adams; +Cc: 12436 "Drew Adams" <drew.adams@oracle.com> writes: > Rename these commands and deprecate the old names. There is no such > thing as a multiframe window, which would mean a window on more than one > frame. These commands should be called something like: > `(next|previous)-window-any-frame'. I agree. Any objections? There does not seem to be many users of these functions: ./frame.el:522:(defun next-multiframe-window () ./frame.el:530:(defun previous-multiframe-window () ./term/ns-win.el:105:(define-key global-map [?\s-'] 'next-multiframe-window) -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#12436: 24.2.50; rename `(next|previous)-multiframe-window' 2016-04-28 17:23 ` Lars Ingebrigtsen @ 2019-10-30 22:30 ` Stefan Kangas 2019-10-30 22:46 ` Lars Ingebrigtsen ` (2 more replies) 0 siblings, 3 replies; 8+ messages in thread From: Stefan Kangas @ 2019-10-30 22:30 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 12436 [-- Attachment #1: Type: text/plain, Size: 671 bytes --] Lars Ingebrigtsen <larsi@gnus.org> writes: > "Drew Adams" <drew.adams@oracle.com> writes: > >> Rename these commands and deprecate the old names. There is no such >> thing as a multiframe window, which would mean a window on more than one >> frame. These commands should be called something like: >> `(next|previous)-window-any-frame'. > > I agree. Any objections? There does not seem to be many users of these > functions: > > ./frame.el:522:(defun next-multiframe-window () > ./frame.el:530:(defun previous-multiframe-window () > ./term/ns-win.el:105:(define-key global-map [?\s-'] 'next-multiframe-window) Makes sense to me. Any objections to the below patch? [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #2: 0001-Rename-multiframe-window-to-window-any-frame.patch --] [-- Type: text/x-diff, Size: 3720 bytes --] From 1ee698b0b0722c18ee211d79821370af2f60af56 Mon Sep 17 00:00:00 2001 From: Stefan Kangas <stefankangas@gmail.com> Date: Wed, 30 Oct 2019 23:27:37 +0100 Subject: [PATCH] Rename `*-multiframe-window' to '*-window-any-frame' Suggested by Drew Adams. * lisp/frame.el (next-multiframe-window): Define as obsolete alias for... (next-window-any-frame): ...new function name. (previous-multiframe-window): Define as obsolete alias for... (previous-window-any-frame): ...new function name. (Bug#12436) * lisp/term/ns-win.el (global-map): Use new command name. * doc/emacs/windows.texi (Other Window): Document it. * etc/NEWS: Announce it. --- doc/emacs/windows.texi | 4 ++-- etc/NEWS | 5 +++++ lisp/frame.el | 8 ++++++-- lisp/term/ns-win.el | 2 +- 4 files changed, 14 insertions(+), 5 deletions(-) diff --git a/doc/emacs/windows.texi b/doc/emacs/windows.texi index 19f6cff7bf..53e9a4bb58 100644 --- a/doc/emacs/windows.texi +++ b/doc/emacs/windows.texi @@ -181,12 +181,12 @@ Other Window back and finish supplying the minibuffer argument that is requested. @xref{Minibuffer Edit}. -@findex next-multiframe-window +@findex next-window-any-frame The @code{other-window} command will normally only switch to the next window in the current frame (unless otherwise configured). If you work in a multi-frame environment and you want windows in all frames to be part of the cycle, you can rebind @kbd{C-x o} to the -@code{next-multiframe-window} command. (@xref{Rebinding}, for how to +@code{next-window-any-frame} command. (@xref{Rebinding}, for how to rebind a command.) @kindex C-M-v diff --git a/etc/NEWS b/etc/NEWS index cd1e46bb87..bb288a4aac 100644 --- a/etc/NEWS +++ b/etc/NEWS @@ -202,6 +202,11 @@ the new version of the file again.) \f * Changes in Emacs 27.1 ++++ +** 'next-multiframe-window' and 'previous-multiframe-window' are obsolete. +Use new names 'next-window-any-frame' and 'previous-window-any-frame' +instead. + ** emacsclient +++ diff --git a/lisp/frame.el b/lisp/frame.el index 92fe57fbe6..7aa1b75f1f 100644 --- a/lisp/frame.el +++ b/lisp/frame.el @@ -673,21 +673,25 @@ get-other-frame is not considered (see `next-frame')." (if (equal (next-frame) (selected-frame)) (make-frame) (next-frame))) -(defun next-multiframe-window () +(defun next-window-any-frame () "Select the next window, regardless of which frame it is on." (interactive) (select-window (next-window (selected-window) (> (minibuffer-depth) 0) 0)) (select-frame-set-input-focus (selected-frame))) +(define-obsolete-function-alias 'next-multiframe-window + 'next-window-any-frame "27.1") -(defun previous-multiframe-window () +(defun previous-window-any-frame () "Select the previous window, regardless of which frame it is on." (interactive) (select-window (previous-window (selected-window) (> (minibuffer-depth) 0) 0)) (select-frame-set-input-focus (selected-frame))) +(define-obsolete-function-alias 'previous-multiframe-window + 'previous-window-any-frame "27.1") (defun window-system-for-display (display) "Return the window system for DISPLAY. diff --git a/lisp/term/ns-win.el b/lisp/term/ns-win.el index 83b845b1a5..1841e679ac 100644 --- a/lisp/term/ns-win.el +++ b/lisp/term/ns-win.el @@ -102,7 +102,7 @@ 'ns-alternatives-map ;; Here are some Nextstep-like bindings for command key sequences. (define-key global-map [?\s-,] 'customize) -(define-key global-map [?\s-'] 'next-multiframe-window) +(define-key global-map [?\s-'] 'next-window-any-frame) (define-key global-map [?\s-`] 'other-frame) (define-key global-map [?\s-~] 'ns-prev-frame) (define-key global-map [?\s--] 'center-line) -- 2.20.1 [-- Attachment #3: Type: text/plain, Size: 29 bytes --] Best regards, Stefan Kangas ^ permalink raw reply related [flat|nested] 8+ messages in thread
* bug#12436: 24.2.50; rename `(next|previous)-multiframe-window' 2019-10-30 22:30 ` Stefan Kangas @ 2019-10-30 22:46 ` Lars Ingebrigtsen 2019-10-30 23:20 ` Drew Adams 2019-10-31 14:05 ` Eli Zaretskii 2 siblings, 0 replies; 8+ messages in thread From: Lars Ingebrigtsen @ 2019-10-30 22:46 UTC (permalink / raw) To: Stefan Kangas; +Cc: 12436 Stefan Kangas <stefan@marxist.se> writes: > Makes sense to me. Any objections to the below patch? > >>From 1ee698b0b0722c18ee211d79821370af2f60af56 Mon Sep 17 00:00:00 2001 > From: Stefan Kangas <stefankangas@gmail.com> > Date: Wed, 30 Oct 2019 23:27:37 +0100 > Subject: [PATCH] Rename `*-multiframe-window' to '*-window-any-frame' > > Suggested by Drew Adams. > > * lisp/frame.el > (next-multiframe-window): Define as obsolete alias for... > (next-window-any-frame): ...new function name. > (previous-multiframe-window): Define as obsolete alias for... > (previous-window-any-frame): ...new function name. (Bug#12436) Looks good to me. -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#12436: 24.2.50; rename `(next|previous)-multiframe-window' 2019-10-30 22:30 ` Stefan Kangas 2019-10-30 22:46 ` Lars Ingebrigtsen @ 2019-10-30 23:20 ` Drew Adams 2019-10-31 14:05 ` Eli Zaretskii 2 siblings, 0 replies; 8+ messages in thread From: Drew Adams @ 2019-10-30 23:20 UTC (permalink / raw) To: Stefan Kangas, Lars Ingebrigtsen; +Cc: 12436 > Any objections to the below patch? Not from me. ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#12436: 24.2.50; rename `(next|previous)-multiframe-window' 2019-10-30 22:30 ` Stefan Kangas 2019-10-30 22:46 ` Lars Ingebrigtsen 2019-10-30 23:20 ` Drew Adams @ 2019-10-31 14:05 ` Eli Zaretskii 2019-11-02 1:37 ` Stefan Kangas 2 siblings, 1 reply; 8+ messages in thread From: Eli Zaretskii @ 2019-10-31 14:05 UTC (permalink / raw) To: Stefan Kangas; +Cc: larsi, 12436 > From: Stefan Kangas <stefan@marxist.se> > Date: Wed, 30 Oct 2019 23:30:53 +0100 > Cc: 12436@debbugs.gnu.org > > >> Rename these commands and deprecate the old names. There is no such > >> thing as a multiframe window, which would mean a window on more than one > >> frame. These commands should be called something like: > >> `(next|previous)-window-any-frame'. > > > > I agree. Any objections? There does not seem to be many users of these > > functions: > > > > ./frame.el:522:(defun next-multiframe-window () > > ./frame.el:530:(defun previous-multiframe-window () > > ./term/ns-win.el:105:(define-key global-map [?\s-'] 'next-multiframe-window) They are commands, not functions. So counting the number of their calls in the code is not an objective method of estimating their popularity. > Makes sense to me. Any objections to the below patch? I don't object, although I do wonder what could be the point of renaming commands which were known under these names for the last 28(!) years. Given their age, perhaps consider not obsoleting the old names: who knows whether someone out there has these names burned into their muscle memory? Thanks. ^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#12436: 24.2.50; rename `(next|previous)-multiframe-window' 2019-10-31 14:05 ` Eli Zaretskii @ 2019-11-02 1:37 ` Stefan Kangas 2019-11-05 23:51 ` Stefan Kangas 0 siblings, 1 reply; 8+ messages in thread From: Stefan Kangas @ 2019-11-02 1:37 UTC (permalink / raw) To: Eli Zaretskii; +Cc: larsi, 12436 [-- Attachment #1: Type: text/plain, Size: 562 bytes --] Eli Zaretskii <eliz@gnu.org> writes: >> Makes sense to me. Any objections to the below patch? > > I don't object, although I do wonder what could be the point of > renaming commands which were known under these names for the last > 28(!) years. Given their age, perhaps consider not obsoleting the old > names: who knows whether someone out there has these names burned into > their muscle memory? Makes sense to me; I've attached an updated patch here. If there are no objections, I intend to push it within a couple of days. Best regards, Stefan Kangas [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #2: 0001-Rename-multiframe-window-to-window-any-frame.patch --] [-- Type: text/x-diff, Size: 3877 bytes --] From b13ea22c9338097560d499872ef0a1f4aea15ba7 Mon Sep 17 00:00:00 2001 From: Stefan Kangas <stefankangas@gmail.com> Date: Wed, 30 Oct 2019 23:27:37 +0100 Subject: [PATCH] Rename `*-multiframe-window' to '*-window-any-frame' Suggested by Drew Adams. * lisp/frame.el (next-multiframe-window): Define as obsolete alias for... (next-window-any-frame): ...new function name. (previous-multiframe-window): Define as obsolete alias for... (previous-window-any-frame): ...new function name. (Bug#12436) * lisp/term/ns-win.el (global-map): Use new command name. * doc/emacs/windows.texi (Other Window): Document it. * etc/NEWS: Announce it. --- doc/emacs/windows.texi | 4 ++-- etc/NEWS | 10 ++++++++++ lisp/frame.el | 7 +++++-- lisp/term/ns-win.el | 2 +- 4 files changed, 18 insertions(+), 5 deletions(-) diff --git a/doc/emacs/windows.texi b/doc/emacs/windows.texi index 19f6cff7bf..53e9a4bb58 100644 --- a/doc/emacs/windows.texi +++ b/doc/emacs/windows.texi @@ -181,12 +181,12 @@ Other Window back and finish supplying the minibuffer argument that is requested. @xref{Minibuffer Edit}. -@findex next-multiframe-window +@findex next-window-any-frame The @code{other-window} command will normally only switch to the next window in the current frame (unless otherwise configured). If you work in a multi-frame environment and you want windows in all frames to be part of the cycle, you can rebind @kbd{C-x o} to the -@code{next-multiframe-window} command. (@xref{Rebinding}, for how to +@code{next-window-any-frame} command. (@xref{Rebinding}, for how to rebind a command.) @kindex C-M-v diff --git a/etc/NEWS b/etc/NEWS index e614197125..e005cc9406 100644 --- a/etc/NEWS +++ b/etc/NEWS @@ -202,6 +202,16 @@ the new version of the file again.) \f * Changes in Emacs 27.1 ++++ +** 'next/previous-multiframe-window' have been renamed. +The new names are as follows: + + 'next-multiframe-window' -> 'next-window-any-frame' + 'previous-multiframe-window' -> 'previous-window-any-frame' + +The old names are maintained as function aliases for backward +compatibility. + ** emacsclient +++ diff --git a/lisp/frame.el b/lisp/frame.el index 92fe57fbe6..1a15a4c9f2 100644 --- a/lisp/frame.el +++ b/lisp/frame.el @@ -673,7 +673,7 @@ get-other-frame is not considered (see `next-frame')." (if (equal (next-frame) (selected-frame)) (make-frame) (next-frame))) -(defun next-multiframe-window () +(defun next-window-any-frame () "Select the next window, regardless of which frame it is on." (interactive) (select-window (next-window (selected-window) @@ -681,7 +681,7 @@ next-multiframe-window 0)) (select-frame-set-input-focus (selected-frame))) -(defun previous-multiframe-window () +(defun previous-window-any-frame () "Select the previous window, regardless of which frame it is on." (interactive) (select-window (previous-window (selected-window) @@ -689,6 +689,9 @@ previous-multiframe-window 0)) (select-frame-set-input-focus (selected-frame))) +(defalias 'next-multiframe-window 'next-window-any-frame) +(defalias 'previous-multiframe-window 'previous-window-any-frame) + (defun window-system-for-display (display) "Return the window system for DISPLAY. Return nil if we don't know how to interpret DISPLAY." diff --git a/lisp/term/ns-win.el b/lisp/term/ns-win.el index 83b845b1a5..1841e679ac 100644 --- a/lisp/term/ns-win.el +++ b/lisp/term/ns-win.el @@ -102,7 +102,7 @@ 'ns-alternatives-map ;; Here are some Nextstep-like bindings for command key sequences. (define-key global-map [?\s-,] 'customize) -(define-key global-map [?\s-'] 'next-multiframe-window) +(define-key global-map [?\s-'] 'next-window-any-frame) (define-key global-map [?\s-`] 'other-frame) (define-key global-map [?\s-~] 'ns-prev-frame) (define-key global-map [?\s--] 'center-line) -- 2.20.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* bug#12436: 24.2.50; rename `(next|previous)-multiframe-window' 2019-11-02 1:37 ` Stefan Kangas @ 2019-11-05 23:51 ` Stefan Kangas 0 siblings, 0 replies; 8+ messages in thread From: Stefan Kangas @ 2019-11-05 23:51 UTC (permalink / raw) To: Eli Zaretskii; +Cc: larsi, 12436 close 12436 27.1 thanks Stefan Kangas <stefan@marxist.se> writes: > Makes sense to me; I've attached an updated patch here. If there are > no objections, I intend to push it within a couple of days. Now pushed to master as commit c91c145d61. Closing. Best regards, Stefan Kangas ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-11-05 23:51 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-09-13 16:30 bug#12436: 24.2.50; rename `(next|previous)-multiframe-window' Drew Adams 2016-04-28 17:23 ` Lars Ingebrigtsen 2019-10-30 22:30 ` Stefan Kangas 2019-10-30 22:46 ` Lars Ingebrigtsen 2019-10-30 23:20 ` Drew Adams 2019-10-31 14:05 ` Eli Zaretskii 2019-11-02 1:37 ` Stefan Kangas 2019-11-05 23:51 ` Stefan Kangas
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).