diff options
| author | Dmitry Antipov | 2013-08-05 18:26:57 +0400 |
|---|---|---|
| committer | Dmitry Antipov | 2013-08-05 18:26:57 +0400 |
| commit | 290d5b5890c5c8d096bd846eb48552d4608d5ed6 (patch) | |
| tree | 2c1e39b6b5f1e82f179ee764eab34d012376076e /lisp/vc | |
| parent | b27a1bd53ac62bad378cbafcfb594ac37357bbc6 (diff) | |
| download | emacs-290d5b5890c5c8d096bd846eb48552d4608d5ed6.tar.gz emacs-290d5b5890c5c8d096bd846eb48552d4608d5ed6.zip | |
Do not call to `selected-window' where it is assumed by default.
Affected functions are `window-minibuffer-p', `window-dedicated-p',
`window-hscroll', `window-width', `window-height', `window-buffer',
`window-frame', `window-start', `window-point', `next-window'
and `window-display-table'.
* abbrev.el (abbrev--default-expand):
* bs.el (bs--show-with-configuration):
* buff-menu.el (Buffer-menu-mouse-select):
* calc/calc.el (calc):
* calendar/calendar.el (calendar-generate-window):
* calendar/diary-lib.el (diary-simple-display, diary-show-all-entries)
(diary-make-entry):
* comint.el (send-invisible, comint-dynamic-complete-filename)
(comint-dynamic-simple-complete, comint-dynamic-list-completions):
* completion.el (complete):
* dabbrev.el (dabbrev-expand, dabbrev--make-friend-buffer-list):
* disp-table.el (describe-current-display-table):
* doc-view.el (doc-view-insert-image):
* ebuff-menu.el (Electric-buffer-menu-mouse-select):
* ehelp.el (with-electric-help):
* emacs-lisp/easy-mmode.el (easy-mmode-define-navigation):
* emacs-lisp/edebug.el (edebug-two-window-p, edebug-pop-to-buffer):
* emacs-lisp/helper.el (Helper-help-scroller):
* emulation/cua-base.el (cua--post-command-handler-1):
* eshell/esh-mode.el (eshell-output-filter):
* ffap.el (ffap-gnus-wrapper):
* help-macro.el (make-help-screen):
* hilit-chg.el (highlight-compare-buffers):
* hippie-exp.el (hippie-expand, try-expand-dabbrev-visible):
* hl-line.el (global-hl-line-highlight):
* icomplete.el (icomplete-simple-completing-p):
* isearch.el (isearch-done):
* jit-lock.el (jit-lock-stealth-fontify):
* mail/rmailsum.el (rmail-summary-scroll-msg-up):
* lisp/mouse-drag.el (mouse-drag-should-do-col-scrolling):
* mpc.el (mpc-tagbrowser, mpc):
* net/rcirc.el (rcirc-any-buffer):
* play/gomoku.el (gomoku-max-width, gomoku-max-height):
* play/landmark.el (landmark-max-width, landmark-max-height):
* play/zone.el (zone):
* progmodes/compile.el (compilation-goto-locus):
* progmodes/ebrowse.el (ebrowse-view/find-file-and-search-pattern):
* progmodes/etags.el (find-tag-other-window):
* progmodes/fortran.el (fortran-column-ruler):
* progmodes/gdb-mi.el (gdb-mouse-toggle-breakpoint-fringe):
* progmodes/verilog-mode.el (verilog-point-text):
* reposition.el (reposition-window):
* rot13.el (toggle-rot13-mode):
* server.el (server-switch-buffer):
* shell.el (shell-dynamic-complete-command)
(shell-dynamic-complete-environment-variable):
* simple.el (insert-buffer, set-selective-display)
(delete-completion-window):
* speedbar.el (speedbar-timer-fn, speedbar-center-buffer-smartly)
(speedbar-recenter):
* startup.el (fancy-splash-head):
* textmodes/ispell.el (ispell-command-loop):
* textmodes/makeinfo.el (makeinfo-compilation-sentinel-region):
* tutorial.el (help-with-tutorial):
* vc/add-log.el (add-change-log-entry):
* vc/compare-w.el (compare-windows):
* vc/ediff-help.el (ediff-indent-help-message):
* vc/ediff-util.el (ediff-setup-control-buffer, ediff-position-region):
* vc/ediff-wind.el (ediff-skip-unsuitable-frames)
(ediff-setup-control-frame):
* vc/emerge.el (emerge-position-region):
* vc/pcvs-util.el (cvs-bury-buffer):
* window.el (walk-windows, mouse-autoselect-window-select):
* winner.el (winner-set-conf, winner-undo): Related users changed.
Diffstat (limited to 'lisp/vc')
| -rw-r--r-- | lisp/vc/add-log.el | 6 | ||||
| -rw-r--r-- | lisp/vc/compare-w.el | 2 | ||||
| -rw-r--r-- | lisp/vc/ediff-help.el | 2 | ||||
| -rw-r--r-- | lisp/vc/ediff-util.el | 4 | ||||
| -rw-r--r-- | lisp/vc/ediff-wind.el | 6 | ||||
| -rw-r--r-- | lisp/vc/emerge.el | 2 | ||||
| -rw-r--r-- | lisp/vc/pcvs-util.el | 6 |
7 files changed, 14 insertions, 14 deletions
diff --git a/lisp/vc/add-log.el b/lisp/vc/add-log.el index f0ea9c68464..f926a164797 100644 --- a/lisp/vc/add-log.el +++ b/lisp/vc/add-log.el | |||
| @@ -813,12 +813,12 @@ non-nil, otherwise in local time." | |||
| 813 | 813 | ||
| 814 | (unless (equal file-name buffer-file-name) | 814 | (unless (equal file-name buffer-file-name) |
| 815 | (cond | 815 | (cond |
| 816 | ((equal file-name (buffer-file-name (window-buffer (selected-window)))) | 816 | ((equal file-name (buffer-file-name (window-buffer))) |
| 817 | ;; If the selected window already shows the desired buffer don't show | 817 | ;; If the selected window already shows the desired buffer don't show |
| 818 | ;; it again (particularly important if other-window is true). | 818 | ;; it again (particularly important if other-window is true). |
| 819 | ;; This is important for diff-add-change-log-entries-other-window. | 819 | ;; This is important for diff-add-change-log-entries-other-window. |
| 820 | (set-buffer (window-buffer (selected-window)))) | 820 | (set-buffer (window-buffer))) |
| 821 | ((or other-window (window-dedicated-p (selected-window))) | 821 | ((or other-window (window-dedicated-p)) |
| 822 | (find-file-other-window file-name)) | 822 | (find-file-other-window file-name)) |
| 823 | (t (find-file file-name)))) | 823 | (t (find-file file-name)))) |
| 824 | (or (derived-mode-p 'change-log-mode) | 824 | (or (derived-mode-p 'change-log-mode) |
diff --git a/lisp/vc/compare-w.el b/lisp/vc/compare-w.el index ae4fcc99a25..235ebe5a072 100644 --- a/lisp/vc/compare-w.el +++ b/lisp/vc/compare-w.el | |||
| @@ -179,7 +179,7 @@ on third call it again advances points to the next difference and so on." | |||
| 179 | 'compare-windows-sync-regexp | 179 | 'compare-windows-sync-regexp |
| 180 | compare-windows-sync))) | 180 | compare-windows-sync))) |
| 181 | (setq p1 (point) b1 (current-buffer)) | 181 | (setq p1 (point) b1 (current-buffer)) |
| 182 | (setq w2 (next-window (selected-window))) | 182 | (setq w2 (next-window)) |
| 183 | (if (eq w2 (selected-window)) | 183 | (if (eq w2 (selected-window)) |
| 184 | (setq w2 (next-window (selected-window) nil 'visible))) | 184 | (setq w2 (next-window (selected-window) nil 'visible))) |
| 185 | (if (eq w2 (selected-window)) | 185 | (if (eq w2 (selected-window)) |
diff --git a/lisp/vc/ediff-help.el b/lisp/vc/ediff-help.el index 833e84b6cea..78d1a6abfd1 100644 --- a/lisp/vc/ediff-help.el +++ b/lisp/vc/ediff-help.el | |||
| @@ -253,7 +253,7 @@ the value of this variable and the variables `ediff-help-message-*' in | |||
| 253 | 253 | ||
| 254 | 254 | ||
| 255 | (defun ediff-indent-help-message () | 255 | (defun ediff-indent-help-message () |
| 256 | (let* ((shift (/ (max 0 (- (window-width (selected-window)) | 256 | (let* ((shift (/ (max 0 (- (window-width) |
| 257 | (ediff-help-message-line-length))) | 257 | (ediff-help-message-line-length))) |
| 258 | 2)) | 258 | 2)) |
| 259 | (str (make-string shift ?\ ))) | 259 | (str (make-string shift ?\ ))) |
diff --git a/lisp/vc/ediff-util.el b/lisp/vc/ediff-util.el index 8f84dfb14ef..2f2c71a71e6 100644 --- a/lisp/vc/ediff-util.el +++ b/lisp/vc/ediff-util.el | |||
| @@ -537,7 +537,7 @@ to invocation.") | |||
| 537 | ;; to reside. | 537 | ;; to reside. |
| 538 | (defun ediff-setup-control-buffer (ctl-buf) | 538 | (defun ediff-setup-control-buffer (ctl-buf) |
| 539 | "Set up window for control buffer." | 539 | "Set up window for control buffer." |
| 540 | (if (window-dedicated-p (selected-window)) | 540 | (if (window-dedicated-p) |
| 541 | (set-buffer ctl-buf) ; we are in control frame but just in case | 541 | (set-buffer ctl-buf) ; we are in control frame but just in case |
| 542 | (switch-to-buffer ctl-buf)) | 542 | (switch-to-buffer ctl-buf)) |
| 543 | (let ((window-min-height 2)) | 543 | (let ((window-min-height 2)) |
| @@ -1629,7 +1629,7 @@ the width of the A/B/C windows." | |||
| 1629 | (setq lines (1+ lines))) | 1629 | (setq lines (1+ lines))) |
| 1630 | ;; And position the beginning on the right line | 1630 | ;; And position the beginning on the right line |
| 1631 | (goto-char beg) | 1631 | (goto-char beg) |
| 1632 | (recenter (/ (1+ (max (- (1- (window-height (selected-window))) | 1632 | (recenter (/ (1+ (max (- (1- (window-height)) |
| 1633 | lines) | 1633 | lines) |
| 1634 | 1) | 1634 | 1) |
| 1635 | ) | 1635 | ) |
diff --git a/lisp/vc/ediff-wind.el b/lisp/vc/ediff-wind.el index 124bdbd26df..5a14c193344 100644 --- a/lisp/vc/ediff-wind.el +++ b/lisp/vc/ediff-wind.el | |||
| @@ -861,7 +861,7 @@ into icons, regardless of the window manager." | |||
| 861 | ;; create a new splittable frame if none is found | 861 | ;; create a new splittable frame if none is found |
| 862 | (defun ediff-skip-unsuitable-frames (&optional ok-unsplittable) | 862 | (defun ediff-skip-unsuitable-frames (&optional ok-unsplittable) |
| 863 | (if (ediff-window-display-p) | 863 | (if (ediff-window-display-p) |
| 864 | (let ((wind-frame (window-frame (selected-window))) | 864 | (let ((wind-frame (window-frame)) |
| 865 | seen-windows) | 865 | seen-windows) |
| 866 | (while (and (not (memq (selected-window) seen-windows)) | 866 | (while (and (not (memq (selected-window) seen-windows)) |
| 867 | (or | 867 | (or |
| @@ -877,7 +877,7 @@ into icons, regardless of the window manager." | |||
| 877 | (setq seen-windows (cons (selected-window) seen-windows)) | 877 | (setq seen-windows (cons (selected-window) seen-windows)) |
| 878 | ;; try new window | 878 | ;; try new window |
| 879 | (other-window 1 t) | 879 | (other-window 1 t) |
| 880 | (setq wind-frame (window-frame (selected-window))) | 880 | (setq wind-frame (window-frame)) |
| 881 | ) | 881 | ) |
| 882 | (if (memq (selected-window) seen-windows) | 882 | (if (memq (selected-window) seen-windows) |
| 883 | ;; fed up, no appropriate frames | 883 | ;; fed up, no appropriate frames |
| @@ -939,7 +939,7 @@ into icons, regardless of the window manager." | |||
| 939 | 939 | ||
| 940 | (setq ctl-frame-iconified-p (ediff-frame-iconified-p ctl-frame)) | 940 | (setq ctl-frame-iconified-p (ediff-frame-iconified-p ctl-frame)) |
| 941 | (select-frame ctl-frame) | 941 | (select-frame ctl-frame) |
| 942 | (if (window-dedicated-p (selected-window)) | 942 | (if (window-dedicated-p) |
| 943 | () | 943 | () |
| 944 | (delete-other-windows) | 944 | (delete-other-windows) |
| 945 | (switch-to-buffer ctl-buffer)) | 945 | (switch-to-buffer ctl-buffer)) |
diff --git a/lisp/vc/emerge.el b/lisp/vc/emerge.el index 0a1bd044125..8d0e8efc75c 100644 --- a/lisp/vc/emerge.el +++ b/lisp/vc/emerge.el | |||
| @@ -1770,7 +1770,7 @@ to the left margin, if they are in windows." | |||
| 1770 | (setq lines (1+ lines))) | 1770 | (setq lines (1+ lines))) |
| 1771 | ;; And position the beginning on the right line | 1771 | ;; And position the beginning on the right line |
| 1772 | (goto-char beg) | 1772 | (goto-char beg) |
| 1773 | (recenter (/ (1+ (- (1- (window-height (selected-window))) | 1773 | (recenter (/ (1+ (- (1- (window-height)) |
| 1774 | lines)) | 1774 | lines)) |
| 1775 | 2)))))) | 1775 | 2)))))) |
| 1776 | (goto-char pos)) | 1776 | (goto-char pos)) |
diff --git a/lisp/vc/pcvs-util.el b/lisp/vc/pcvs-util.el index 28ae4ed3de3..2868bb98b29 100644 --- a/lisp/vc/pcvs-util.el +++ b/lisp/vc/pcvs-util.el | |||
| @@ -97,7 +97,7 @@ try to split a new window instead." | |||
| 97 | BUF is assumed to be a temporary buffer used from the buffer MAINBUF." | 97 | BUF is assumed to be a temporary buffer used from the buffer MAINBUF." |
| 98 | (interactive (list (current-buffer))) | 98 | (interactive (list (current-buffer))) |
| 99 | (save-current-buffer | 99 | (save-current-buffer |
| 100 | (let ((win (if (eq buf (window-buffer (selected-window))) (selected-window) | 100 | (let ((win (if (eq buf (window-buffer)) (selected-window) |
| 101 | (get-buffer-window buf t)))) | 101 | (get-buffer-window buf t)))) |
| 102 | (when win | 102 | (when win |
| 103 | (if (window-dedicated-p win) | 103 | (if (window-dedicated-p win) |
| @@ -111,8 +111,8 @@ BUF is assumed to be a temporary buffer used from the buffer MAINBUF." | |||
| 111 | ;;; ) | 111 | ;;; ) |
| 112 | ))) | 112 | ))) |
| 113 | (with-current-buffer buf | 113 | (with-current-buffer buf |
| 114 | (bury-buffer (unless (and (eq buf (window-buffer (selected-window))) | 114 | (bury-buffer (unless (and (eq buf (window-buffer)) |
| 115 | (not (window-dedicated-p (selected-window)))) | 115 | (not (window-dedicated-p))) |
| 116 | buf))) | 116 | buf))) |
| 117 | (when mainbuf | 117 | (when mainbuf |
| 118 | (let ((mainwin (or (get-buffer-window mainbuf) | 118 | (let ((mainwin (or (get-buffer-window mainbuf) |