]> code.delx.au - gnu-emacs/blobdiff - lisp/ediff-wind.el
Comment changes.
[gnu-emacs] / lisp / ediff-wind.el
index 073160e5c3349faf54706d912e40e456a834254a..ae65b1d6f51131ce31fd4afad0f04017eecf731e 100644 (file)
@@ -1,8 +1,8 @@
 ;;; ediff-wind.el --- window manipulation utilities
 
-;; Copyright (C) 1994, 1995, 1996, 1997 Free Software Foundation, Inc.
+;; Copyright (C) 1994, 95, 96, 97, 2000, 01, 02 Free Software Foundation, Inc.
 
-;; Author: Michael Kifer <kifer@cs.sunysb.edu>
+;; Author: Michael Kifer <kifer@cs.stonybrook.edu>
 
 ;; This file is part of GNU Emacs.
 
@@ -21,6 +21,8 @@
 ;; Free Software Foundation, Inc., 59 Temple Place - Suite 330,
 ;; Boston, MA 02111-1307, USA.
 
+;;; Commentary:
+
 ;;; Code:
         
 (provide 'ediff-wind)
@@ -34,6 +36,7 @@
 (defvar left-toolbar-width)
 (defvar right-toolbar-width)
 (defvar default-menubar)
+(defvar top-gutter)
 (defvar frame-icon-title-format)
 (defvar ediff-diff-status)
 (defvar ediff-emacs-p)
@@ -91,7 +94,9 @@ The basic guidelines:
        Buffer C may not be used in jobs that compare only two buffers.
 If you plan to do something fancy, take a close look at how the two
 provided functions are written."
-  :type 'function
+  :type '(choice (const :tag "Multi Frame" ediff-setup-windows-multiframe)
+                (const :tag "Single Frame" ediff-setup-windows-plain)
+                (function :tag "Other function"))
   :group 'ediff-window)
 
 ;; indicates if we are in a multiframe setup
@@ -151,6 +156,7 @@ In this case, Ediff will use those frames to display these buffers."
    '(vertical-scroll-bars . nil)  ; Emacs only
    '(scrollbar-width . 0)         ; XEmacs only
    '(menu-bar-lines . 0)          ; Emacs only
+   '(tool-bar-lines . 0)          ; Emacs 21+ only
    ;; don't lower and auto-raise
    '(auto-lower . nil)
    '(auto-raise . t)
@@ -268,35 +274,39 @@ into icons, regardless of the window manager."
          (beep 1))
       (message "Please click on Window %d " wind-number))
     (ediff-read-event) ; discard event
-    (setq wind (if ediff-xemacs-p
-                  (event-window event)
-                (posn-window (event-start event))))
+    (setq wind (ediff-cond-compile-for-xemacs-or-emacs
+               (event-window event) ; xemacs
+               (posn-window (event-start event)) ; emacs
+               )
+         )
     ))
       
 
+;; Select the lowest window on the frame.
 (defun ediff-select-lowest-window ()
-  "Select the lowest window on the frame."
-  (if ediff-xemacs-p
-      (select-window (frame-lowest-window))
-    (let* ((lowest-window (selected-window))
-          (bottom-edge (car (cdr (cdr (cdr (window-edges))))))
-          (last-window (save-excursion
-                         (other-window -1) (selected-window)))
-          (window-search t))
-      (while window-search
-       (let* ((this-window (next-window))
-              (next-bottom-edge
-               (car (cdr (cdr (cdr (window-edges this-window)))))))
-         (if (< bottom-edge next-bottom-edge)
-             (progn
-               (setq bottom-edge next-bottom-edge)
-               (setq lowest-window this-window)))
-         
-         (select-window this-window)
-         (if (eq last-window this-window)
-             (progn
-               (select-window lowest-window)
-               (setq window-search nil))))))))
+  (ediff-cond-compile-for-xemacs-or-emacs
+   (select-window (frame-lowest-window)) ; xemacs
+   ;; emacs
+   (let* ((lowest-window (selected-window))
+         (bottom-edge (car (cdr (cdr (cdr (window-edges))))))
+         (last-window (save-excursion
+                        (other-window -1) (selected-window)))
+         (window-search t))
+     (while window-search
+       (let* ((this-window (next-window))
+             (next-bottom-edge
+              (car (cdr (cdr (cdr (window-edges this-window)))))))
+        (if (< bottom-edge next-bottom-edge)
+            (progn
+              (setq bottom-edge next-bottom-edge)
+              (setq lowest-window this-window)))
+        
+        (select-window this-window)
+        (if (eq last-window this-window)
+            (progn
+              (select-window lowest-window)
+              (setq window-search nil))))))
+   ))
 
 
 ;;; Common window setup routines
@@ -830,25 +840,30 @@ into icons, regardless of the window manager."
 ;; create a new splittable frame if none is found
 (defun ediff-skip-unsuitable-frames (&optional ok-unsplittable)
   (if (ediff-window-display-p)
-      (let (last-window)
-       (while (and (not (eq (selected-window) last-window))
+      ;;(let (last-window)
+      (let (seen-windows)
+       ;; (memq ... seen-windows) has quadratic behavior,
+       ;; but (eq ... last-window) runs into an emacs bug where next-window
+       ;; gets stuck in a loop if the only frame is the minibuffer.
+       ;;
+       ;;(while (and (not (eq (selected-window) last-window))
+       (while (and (not (memq (selected-window) seen-windows))
                    (or
                     (ediff-frame-has-dedicated-windows (selected-frame))
                     (ediff-frame-iconified-p (selected-frame))
-                    ;; skip small frames
-                    (< (frame-height (selected-frame))
-                       (* 3 window-min-height))
                     ;; skip small windows
-                    (< (window-height (selected-window))
+                    (< (frame-height (selected-frame))
                        (* 3 window-min-height))
                     (if ok-unsplittable
                         nil
                       (ediff-frame-unsplittable-p (selected-frame)))))
          ;; remember where started
-         (or last-window (setq last-window (selected-window)))
+         ;;(or last-window (setq last-window (selected-window)))
+         (setq seen-windows (cons (selected-window) seen-windows))
          ;; try new window
          (other-window 1 t))
-       (if (eq (selected-window) last-window)
+       ;;(if (eq (selected-window) last-window)
+       (if (memq (selected-window) seen-windows)
            ;; fed up, no appropriate frame
            (progn
              (select-frame (make-frame '((unsplittable)))))))))
@@ -888,7 +903,10 @@ into icons, regardless of the window manager."
        fheight fwidth adjusted-parameters)
        
     (ediff-with-current-buffer ctl-buffer
-      (if ediff-xemacs-p (set-buffer-menubar nil))
+      (ediff-cond-compile-for-xemacs-or-emacs
+       (set-buffer-menubar nil) ; xemacs
+       nil ; emacs
+       )
       ;;(setq user-grabbed-mouse (ediff-user-grabbed-mouse))
       (run-hooks 'ediff-before-setup-control-frame-hook))
   
@@ -897,7 +915,16 @@ into icons, regardless of the window manager."
       (setq ctl-frame (if (frame-live-p old-ctl-frame)
                          old-ctl-frame
                        (make-frame ediff-control-frame-parameters))
-           ediff-control-frame ctl-frame))
+           ediff-control-frame ctl-frame)
+      ;; protect against undefined face-attribute
+      (condition-case nil
+         (ediff-cond-compile-for-xemacs-or-emacs
+          nil ; xemacs
+          (when (face-attribute 'mode-line :box)
+            (set-face-attribute 'mode-line ctl-frame :box nil))
+          )
+       (error))
+      )
     
     (setq ctl-frame-iconified-p (ediff-frame-iconified-p ctl-frame))
     (select-frame ctl-frame)
@@ -941,14 +968,19 @@ into icons, regardless of the window manager."
     ;; In XEmacs, buffer menubar needs to be killed before frame parameters
     ;; are changed.
     (if (ediff-has-toolbar-support-p)
-       (progn
-         (set-specifier top-toolbar-height (list ctl-frame 2))
-         (sit-for 0)
-         (set-specifier top-toolbar-height (list ctl-frame 0))
-         ;;(set-specifier bottom-toolbar-height (list ctl-frame 0))
-         (set-specifier left-toolbar-width (list ctl-frame 0))
-         (set-specifier right-toolbar-width (list ctl-frame 0))
-         ))
+       (ediff-cond-compile-for-xemacs-or-emacs
+        (progn ; xemacs
+          (if (ediff-has-gutter-support-p)
+              (set-specifier top-gutter (list ctl-frame nil)))
+          (sit-for 0)
+          (set-specifier top-toolbar-height (list ctl-frame 0))
+          ;;(set-specifier bottom-toolbar-height (list ctl-frame 0))
+          (set-specifier left-toolbar-width (list ctl-frame 0))
+          (set-specifier right-toolbar-width (list ctl-frame 0))
+          )
+        nil ; emacs
+        )
+      )
     
     ;; Under OS/2 (emx) we have to call modify frame parameters twice, in order
     ;; to make sure that at least once we do it for non-iconified frame.  If
@@ -1004,8 +1036,12 @@ into icons, regardless of the window manager."
        
     (if ediff-xemacs-p
        (ediff-with-current-buffer ctl-buffer
-         (make-local-hook 'select-frame-hook)
-         (add-hook 'select-frame-hook 'ediff-xemacs-select-frame-hook nil t)
+         (ediff-cond-compile-for-xemacs-or-emacs
+          (make-local-hook 'select-frame-hook) ; xemacs
+          nil     ; emacs
+          )
+         (add-hook
+          'select-frame-hook 'ediff-xemacs-select-frame-hook nil 'local)
          ))
        
     (ediff-with-current-buffer ctl-buffer
@@ -1017,8 +1053,10 @@ into icons, regardless of the window manager."
   (ediff-with-current-buffer ctl-buffer
     (if (and (ediff-window-display-p) (frame-live-p ediff-control-frame))
        (let ((ctl-frame ediff-control-frame))
-         (if ediff-xemacs-p
-             (set-buffer-menubar default-menubar))
+         (ediff-cond-compile-for-xemacs-or-emacs
+          (set-buffer-menubar default-menubar) ; xemacs
+          nil ; emacs
+          )
          (setq ediff-control-frame nil)
          (delete-frame ctl-frame)
          )))