Add arch taglines
[bpt/emacs.git] / lisp / iswitchb.el
index 18142e6..d8f2a5c 100644 (file)
@@ -1,10 +1,10 @@
 ;;; iswitchb.el --- switch between buffers using substrings
 
-;; Copyright (C) 1996, 1997  Free Software Foundation, Inc.
+;; Copyright (C) 1996, 1997, 2000, 2001, 2003  Free Software Foundation, Inc.
 
-;; Author: Stephen Eglen <stephen@anc.ed.ac.uk>
-;; Maintainer: Stephen Eglen <stephen@anc.ed.ac.uk>
-;; Keywords: extensions convenience
+;; Author: Stephen Eglen <stephen@gnu.org>
+;; Maintainer: Stephen Eglen <stephen@gnu.org>
+;; Keywords: completion convenience
 ;; location: http://www.anc.ed.ac.uk/~stephen/emacs/
 
 ;; This file is part of GNU Emacs.
 ;;; Commentary:
 
 ;; Installation:
-;; To get the functions in this package bound to keys, do
-;; (iswitchb-default-keybindings)
-;;
-;; If you want to use the features of iswitchb, but without rebinding
-;; the keys as above, then you need to add the following hook:
-;; (add-hook 'minibuffer-setup-hook 'iswitchb-minibuffer-setup)
+;; To get the functions in this package bound to keys, use
+;; M-x iswitchb-mode or customize the option `iswitchb-mode'.
+;; Alternatively, add the following line to your .emacs:
+;; (iswitchb-mode 1)
 
 ;; As you type in a substring, the list of buffers currently matching
-;; the substring are displayed as you type.  The list is ordered so
+;; the substring is displayed as you type.  The list is ordered so
 ;; that the most recent buffers visited come at the start of the list.
 ;; The buffer at the start of the list will be the one visited when
 ;; you press return.  By typing more of the substring, the list is
 ;; narrowed down so that gradually the buffer you want will be at the
-;; top of the list.  Alternatively, you can use C-s an C-r to rotate
+;; top of the list.  Alternatively, you can use C-s and C-r to rotate
 ;; buffer names in the list until the one you want is at the top of
 ;; the list.  Completion is also available so that you can see what is
 ;; common to all of the matching buffers as you type.
 
 ;; This code is similar to a couple of other packages.  Michael R Cook
-;; <mcook@cognex.com> wrote a similar buffer switching package, but
+;; <cook@sightpath.com> wrote a similar buffer switching package, but
 ;; does exact matching rather than substring matching on buffer names.
 ;; I also modified a couple of functions from icomplete.el to provide
 ;; the completion feedback in the minibuffer.
 
-;;; Example 
+;;; Example
 
-;;If I have two buffers called "123456" and "123", with "123456" the
-;;most recent, when I use iswitchb, I first of all get presented with
-;;the list of all the buffers
+;; If I have two buffers called "123456" and "123", with "123456" the
+;; most recent, when I use iswitchb, I first of all get presented with
+;; the list of all the buffers
 ;;
-;;       iswitch  {123456,123} 
+;;       iswitch  {123456,123}
 ;;
 ;; If I then press 2:
 ;;       iswitch 2[3]{123456,123}
 ;; The list in {} are the matching buffers, most recent first (buffers
 ;; visible in the current frame are put at the end of the list by
 ;; default).  At any time I can select the item at the head of the
-;; list by pressing RET.  I can also bring the put the first element
-;; at the end of the list by pressing C-s, or put the last element at
-;; the head of the list by pressing C-r.  The item in [] indicates
-;; what can be added to my input by pressing TAB.  In this case, I
-;; will get "3" added to my input.  So, press TAB: 
+;; list by pressing RET.  I can also put the first element at the end
+;; of the list by pressing C-s, or put the last element at the head of
+;; the list by pressing C-r.  The item in [] indicates what can be
+;; added to my input by pressing TAB.  In this case, I will get "3"
+;; added to my input.  So, press TAB:
 ;;      iswitch 23{123456,123}
 ;;
 ;; At this point, I still have two matching buffers.
 ;; wanted the second in the list, I could press C-s to move it to the
 ;; top of the list and then RET to select it.
 ;;
-;;However, If I type 4, I only have one match left:
+;; However, if I type 4, I only have one match left:
 ;;       iswitch 234[123456] [Matched]
 ;;
-;;Since there is only one matching buffer left, it is given in [] and we
-;;see the text [Matched] afterwards.  I can now press TAB or RET to go
-;;to that buffer.
+;; Since there is only one matching buffer left, it is given in [] and we
+;; see the text [Matched] afterwards.  I can now press TAB or RET to go
+;; to that buffer.
 ;;
 ;; If however, I now type "a":
 ;;       iswitch 234a [No match]
 ;; If you find that the file you are after is not in a buffer, you can
 ;; press C-x C-f to immediately drop into find-file.
 
+;; See the doc string of iswitchb for full keybindings and features.
+;; (describe-function 'iswitchb)
+
+;; Case matching: The case of strings when matching can be ignored or
+;; used depending on the value of iswitchb-case (default is the same
+;; as case-fold-search, normally t).  Imagine you have the following
+;; buffers:
+;;
+;; INBOX *info* *scratch*
+;;
+;; Then these will be the matching buffers, depending on how you type
+;; the two letters `in' and the value of iswitchb-case:
 ;;
-;;  See the doc string of iswitchb for full keybindings and features.
-;;  (describe-function 'iswitchb)
+;; iswitchb-case   user input  | matching buffers
+;; ----------------------------------------------
+;; nil             in          | *info*
+;; t               in          | INBOX, *info*
+;; t               IN          | INBOX
+;; t               In          | [No match]
 
 ;;; Customisation
 
 ;; See the User Variables section below for easy ways to change the
 ;; functionality of the program.  These are accessible using the
 ;; custom package.
-;; To modify the keybindings, use the hook provided.  For example:
-;;(add-hook 'iswitchb-define-mode-map-hook
-;;       'iswitchb-my-keys)
+;; To modify the keybindings, use something like:
 ;;
+;;(add-hook 'iswitchb-mode-hook 'iswitchb-my-keys)
 ;;(defun iswitchb-my-keys ()
 ;;  "Add my keybindings for iswitchb."
-;;  (define-key iswitchb-mode-map " " 'iswitchb-next-match)
-;;  )
+;;  (define-key iswitchb-mode-map " " 'iswitchb-next-match))
 ;;
 ;; Seeing all the matching buffers
 ;;
 ;; If you have many matching buffers, they may not all fit onto one
-;; line of the minibuffer.  In this case, you should use rsz-mini
-;; (resize-minibuffer-mode).  You can also limit iswitchb so that it
+;; line of the minibuffer.  In Emacs 21, the variable
+;; `resize-mini-windows' controls how many lines of the minibuffer can
+;; be seen.  For older versions of emacs, you can use
+;; `resize-minibuffer-mode'.  You can also limit iswitchb so that it
 ;; only shows a certain number of lines -- see the documentation for
 ;; `iswitchb-minibuffer-setup-hook'.
 
 ;; for the normal buffer selection routine `read-buffer'.  To use
 ;; iswitch for all buffer selections in Emacs, add:
 ;; (setq read-buffer-function 'iswitchb-read-buffer)
-;; (This variable should be present in Emacs 20.3+)
+;; (This variable was introduced in Emacs 20.3.)
 ;; XEmacs users can get the same behaviour by doing:
-;; (defalias 'read-buffer 'iswitchb-read-buffer) 
+;; (defalias 'read-buffer 'iswitchb-read-buffer)
 ;; since `read-buffer' is defined in lisp.
 
+;; Using iswitchb for other completion tasks.
+
+;; Kin Cho (kin@neoscale.com) sent the following suggestion to use
+;; iswitchb for other completion tasks.
+;;
+;; (defun my-icompleting-read (prompt choices)
+;;   "Use iswitch as a completing-read replacement to choose from
+;; choices.  PROMPT is a string to prompt with.  CHOICES is a list of
+;; strings to choose from."
+;;   (let ((iswitchb-make-buflist-hook
+;;          (lambda ()
+;;            (setq iswitchb-temp-buflist choices))))
+;;     (iswitchb-read-buffer prompt)))
+;;
+;; example:
+;; (my-icompleting-read "Which fruit? " '
+;;                  ("apple" "pineapple" "pear" "bananas" "oranges") )
+
+;; Kin Cho also suggested the following defun.  Once you have a subset of
+;; matching buffers matching your current prompt, you can then press
+;; e.g. C-o to restrict matching to those buffers and clearing the prompt:
+;; (defun iswitchb-exclude-nonmatching()
+;;    "Make iswitchb work on only the currently matching names."
+;;    (interactive)
+;;    (setq iswitchb-buflist iswitchb-matches)
+;;    (setq iswitchb-rescan t)
+;;    (delete-minibuffer-contents))
+;;
+;; (add-hook 'iswitchb-define-mode-map-hook
+;;       '(lambda () (define-key
+;;                     iswitchb-mode-map "\C-o"
+;;                     'iswitchb-exclude-nonmatching)))
+
+;; Other lisp packages extend iswitchb behaviour to other tasks.  See
+;; ido.el (by Kim Storm) and mcomplete.el (Yuji Minejima).
+
+;; Window managers: Switching frames/focus follows mouse; Sawfish.
+
+;; If you switch to a buffer that is visible in another frame,
+;; iswitchb can switch focus to that frame.  If your window manager
+;; uses "click to focus" policy for window selection, you should also
+;; set focus-follows-mouse to nil.
+
+;; iswitch functionality has also been implemented for switching
+;; between windows in the Sawfish window manager.
+
 ;; Regexp matching
 
 ;; There is limited provision for regexp matching within iswitchb,
     ;; We have the old custom-library, hack around it!
     (defmacro defgroup (&rest args)
       nil)
-    (defmacro defcustom (var value doc &rest args) 
+    (defmacro defcustom (var value doc &rest args)
       `(defvar ,var ,value ,doc))))
 
 ;;; User Variables
 
 (defgroup iswitchb nil
   "Switch between buffers using substrings."
-  :group 'extensions
   :group 'convenience
-  ;; These links are to be added in later versions of custom and
-  ;; so are currently commented out.
+  :group 'completion
   :link '(emacs-commentary-link :tag "Commentary" "iswitchb.el")
+  :link '(url-link "http://www.anc.ed.ac.uk/~stephen/emacs/")
   :link '(emacs-library-link :tag "Lisp File" "iswitchb.el"))
 
 (defcustom iswitchb-case case-fold-search
-  "*Non-nil if searching of buffer names should ignore case."
+  "*Non-nil if searching of buffer names should ignore case.
+If this is non-nil but the user input has any upper case letters, matching
+is temporarily case sensitive."
   :type 'boolean
   :group 'iswitchb)
 
 For example, traditional behavior is not to list buffers whose names begin
 with a space, for which the regexp is `^ '.  See the source file for
 example functions that filter buffernames."
-  :type '(repeat regexp)
+  :type '(repeat (choice regexp function))
+  :group 'iswitchb)
+
+(defcustom iswitchb-cannot-complete-hook 'iswitchb-completion-help
+  "*Hook run when `iswitchb-complete' can't complete any more.
+The most useful values are `iswitchb-completion-help', which pops up a
+window with completion alternatives, or `iswitchb-next-match' or
+`iswitchb-prev-match', which cycle the buffer list."
+  :type 'hook
   :group 'iswitchb)
 
 ;;; Examples for setting the value of iswitchb-buffer-ignore
@@ -259,10 +328,10 @@ Possible values:
                frame or in the other frame.
 `always-frame'  If a buffer is visible in another frame, raise that
                frame.  Otherwise, visit the buffer in the same window."
-    :type '(choice (const samewindow) 
+    :type '(choice (const samewindow)
                   (const otherwindow)
                   (const display)
-                  (const otherframe) 
+                  (const otherframe)
                   (const maybe-frame)
                   (const always-frame))
     :group 'iswitchb)
@@ -285,18 +354,18 @@ See also `iswitchb-newbuffer'."
   :type 'boolean
   :group 'iswitchb)
 
-(defcustom iswitchb-define-mode-map-hook  nil
-  "*Hook to define keys in `iswitchb-mode-map' for extra keybindings."
-  :type 'hook
-  :group 'iswitchb)
-
 (defcustom iswitchb-use-fonts t
   "*Non-nil means use font-lock fonts for showing first match."
   :type 'boolean
   :group 'iswitchb)
 
+(defcustom iswitchb-use-frame-buffer-list nil
+  "*Non-nil means use the currently selected frame's buffer list."
+  :type 'boolean
+  :group 'iswitchb)
+
 (defcustom iswitchb-make-buflist-hook  nil
-  "*Hook to run when list of matching buffers is created."
+  "Hook to run when list of matching buffers is created."
   :type 'hook
   :group 'iswitchb)
 
@@ -305,20 +374,21 @@ See also `iswitchb-newbuffer'."
 See documentation of `walk-windows' for useful values.")
 
 (defcustom iswitchb-minibuffer-setup-hook nil
-  "*Iswitchb-specific customization of minibuffer setup.
+  "Iswitchb-specific customization of minibuffer setup.
 
 This hook is run during minibuffer setup iff `iswitchb' will be active.
 It is intended for use in customizing iswitchb for interoperation
-with other packages.  For instance:
+with other packages."
+;;;   For instance:
 
-  \(add-hook 'iswitchb-minibuffer-setup-hook 
-           \(function
-            \(lambda ()
-              \(make-local-variable 'resize-minibuffer-window-max-height)
-              \(setq resize-minibuffer-window-max-height 3))))
+;;;   \(add-hook 'iswitchb-minibuffer-setup-hook
+;;;        \(function
+;;;         \(lambda ()
+;;;           \(make-local-variable 'resize-minibuffer-window-max-height)
+;;;           \(setq resize-minibuffer-window-max-height 3))))
 
-will constrain rsz-mini to a maximum minibuffer height of 3 lines when
-iswitchb is running.  Copied from `icomplete-minibuffer-setup-hook'."
+;;; will constrain rsz-mini to a maximum minibuffer height of 3 lines when
+;;; iswitchb is running.  Copied from `icomplete-minibuffer-setup-hook'."
   :type 'hook
   :group 'iswitchb)
 
@@ -327,7 +397,7 @@ iswitchb is running.  Copied from `icomplete-minibuffer-setup-hook'."
 ;;; Internal Variables
 
 (defvar iswitchb-method nil
-  "Stores the method for viewing the selected buffer.  
+  "Stores the method for viewing the selected buffer.
 Its value is one of `samewindow', `otherwindow', `display', `otherframe',
 `maybe-frame' or `always-frame'.  See `iswitchb-default-method' for
 details of values.")
@@ -346,11 +416,11 @@ at the end of the list.  Created by `iswitchb-make-buflist'.")
 ;; todo -- is this necessary?
 
 (defvar iswitchb-use-mycompletion nil
-  "Non-nil means use `iswitchb-buffer' completion feedback.  
+  "Non-nil means use `iswitchb-buffer' completion feedback.
 Should only be set to t by iswitchb functions, so that it doesn't
 interfere with other minibuffer usage.")
 
-(defvar iswitchb-change-word-sub nil 
+(defvar iswitchb-change-word-sub nil
   "Private variable used by `iswitchb-word-matching-substring'.")
 
 (defvar iswitchb-common-match-string  nil
@@ -365,23 +435,46 @@ interfere with other minibuffer usage.")
 (defvar iswitchb-matches nil
   "List of buffers currently matching `iswitchb-text'.")
 
-(defvar iswitchb-mode-map nil
-  "Keymap for `iswitchb-buffer'.")
-
-(defvar  iswitchb-history nil
+(defvar iswitchb-mode-map
+  (let ((map (make-sparse-keymap)))
+    (set-keymap-parent map minibuffer-local-map)
+    (define-key map "?" 'iswitchb-completion-help)
+    (define-key map "\C-s" 'iswitchb-next-match)
+    (define-key map "\C-r" 'iswitchb-prev-match)
+    (define-key map "\t" 'iswitchb-complete)
+    (define-key map "\C-j" 'iswitchb-select-buffer-text)
+    (define-key map "\C-t" 'iswitchb-toggle-regexp)
+    (define-key map "\C-x\C-f" 'iswitchb-find-file)
+    (define-key map "\C-c" 'iswitchb-toggle-case)
+    (define-key map "\C-k" 'iswitchb-kill-buffer)
+    (define-key map "\C-m" 'iswitchb-exit-minibuffer)
+    map)
+  "Minibuffer keymap for `iswitchb-buffer'.")
+
+(defvar iswitchb-global-map
+  (let ((map (make-sparse-keymap)))
+    (substitute-key-definition 'switch-to-buffer ; normally C-x b
+                              'iswitchb-buffer map global-map)
+    (substitute-key-definition 'switch-to-buffer-other-window ; C-x 4 b
+                              'iswitchb-buffer-other-window map global-map)
+    (substitute-key-definition 'switch-to-buffer-other-frame ; C-x 5 b
+                              'iswitchb-buffer-other-frame map global-map)
+    (substitute-key-definition 'display-buffer ; C-x 4 C-o
+                              'iswitchb-display-buffer map global-map)
+    map)
+  "Global keymap for `iswtichb-mode'.")
+
+(defvar iswitchb-history nil
   "History of buffers selected using `iswitchb-buffer'.")
 
-(defvar iswitchb-exit nil 
-  "Flag to monitor how `iswitchb-buffer' exits.  
+(defvar iswitchb-exit nil
+  "Flag to monitor how `iswitchb-buffer' exits.
 If equal to `takeprompt', we use the prompt as the buffer name to be
 selected.")
 
 (defvar iswitchb-buffer-ignore-orig nil
   "Stores original value of `iswitchb-buffer-ignore'.")
 
-(defvar iswitchb-xemacs  (string-match "XEmacs" (emacs-version))
-  "Non-nil if we are running XEmacs.  Otherwise, assume we are running Emacs.")
-
 (defvar iswitchb-default nil
   "Default buffer for iswitchb.")
 
@@ -395,11 +488,16 @@ selected.")
 (defvar iswitchb-bufs-in-frame nil
   "List of the buffers visible in the current frame.")
 
+(defvar iswitchb-minibuf-depth nil
+  "Value we expect to be returned by `minibuffer-depth' in the minibuffer.")
+
 ;;; FUNCTIONS
 
-;;; ISWITCHB KEYMAP 
+;;; ISWITCHB KEYMAP
 (defun iswitchb-define-mode-map ()
-  "Set up the keymap for `iswitchb-buffer'."
+  "Set up the keymap for `iswitchb-buffer'.
+This is obsolete.  Use \\[iswitchb-mode] or customize the
+variable `iswitchb-mode'."
   (interactive)
   (let (map)
     ;; generated every time so that it can inherit new functions.
@@ -419,7 +517,7 @@ selected.")
     (define-key map "\C-m" 'iswitchb-exit-minibuffer)
     (setq iswitchb-mode-map map)
     (run-hooks 'iswitchb-define-mode-map-hook)))
-  
+
 ;;; MAIN FUNCTION
 (defun iswitchb ()
   "Switch to buffer matching a substring.
@@ -437,24 +535,20 @@ If no buffer is found, prompt for a new one.
 
 \\[iswitchb-next-match] Put the first element at the end of the list.
 \\[iswitchb-prev-match] Put the last element at the start of the list.
-\\[iswitchb-complete] Complete a common suffix to the current string that 
+\\[iswitchb-complete] Complete a common suffix to the current string that
 matches all buffers.  If there is only one match, select that buffer.
 If there is no common suffix, show a list of all matching buffers
 in a separate window.
 \\[iswitchb-toggle-regexp] Toggle regexp searching.
 \\[iswitchb-toggle-case] Toggle case-sensitive searching of buffer names.
 \\[iswitchb-completion-help] Show list of matching buffers in separate window.
-\\[iswitchb-find-file] Exit iswitchb and drop into find-file.
+\\[iswitchb-find-file] Exit iswitchb and drop into `find-file'.
 \\[iswitchb-kill-buffer] Kill buffer at head of buffer list."
   ;;\\[iswitchb-toggle-ignore] Toggle ignoring certain buffers (see \
   ;;`iswitchb-buffer-ignore')
-       
-  (let
-      (prompt buf)
-    
-    (setq prompt (format "iswitch "))
 
-    (setq buf (iswitchb-read-buffer prompt))
+  (let* ((prompt "iswitch ")
+        (buf (iswitchb-read-buffer prompt)))
 
     ;;(message "chosen text %s" iswitchb-final-text)
     ;; Choose the buffer name: either the text typed in, or the head
@@ -478,7 +572,7 @@ in a separate window.
 ;;;###autoload
 (defun iswitchb-read-buffer (prompt &optional default require-match)
   "Replacement for the built-in `read-buffer'.
-Return the name of a buffer selected.  
+Return the name of a buffer selected.
 PROMPT is the prompt to give to the user.  DEFAULT if given is the default
 buffer to be selected, which will go to the front of the list.
 If REQUIRE-MATCH is non-nil, an existing-buffer must be selected."
@@ -502,32 +596,38 @@ If REQUIRE-MATCH is non-nil, an existing-buffer must be selected."
            default))
     (iswitchb-make-buflist iswitchb-default)
     (iswitchb-set-matches)
-    (let 
+    (let
        ((minibuffer-local-completion-map iswitchb-mode-map)
-        (iswitchb-prepost-hooks t)
+        ;; Record the minibuffer depth that we expect to find once
+        ;; the minibuffer is set up and iswitchb-entryfn-p is called.
+        (iswitchb-minibuf-depth (1+ (minibuffer-depth)))
         (iswitchb-require-match require-match))
       ;; prompt the user for the buffer name
-      (setq iswitchb-final-text (completing-read 
+      (setq iswitchb-final-text (completing-read
                                 prompt ;the prompt
-                                '(("dummy".1)) ;table
+                                '(("dummy" . 1)) ;table
                                 nil    ;predicate
                                 nil    ;require-match [handled elsewhere]
                                 nil    ;initial-contents
                                 'iswitchb-history)))
+    (if (get-buffer iswitchb-final-text)
+       ;; This happens for example if the buffer was chosen with the mouse.
+       (setq iswitchb-matches (list iswitchb-final-text)))
+
     ;; Handling the require-match must be done in a better way.
     (if (and require-match (not (iswitchb-existing-buffer-p)))
-       (error "must specify valid buffer"))
+       (error "Must specify valid buffer"))
 
-    (if (or 
+    (if (or
         (eq iswitchb-exit 'takeprompt)
         (null iswitchb-matches))
        (setq buf-sel iswitchb-final-text)
       ;; else take head of list
       (setq buf-sel (car iswitchb-matches)))
-    
+
     ;; Or possibly choose the default buffer
-    (if  (equal iswitchb-final-text "")        
-       (setq buf-sel 
+    (if  (equal iswitchb-final-text "")
+       (setq buf-sel
              (car iswitchb-matches)))
 
     buf-sel))
@@ -558,12 +658,12 @@ The result is stored in `iswitchb-common-match-string'."
   (interactive)
   (let (res)
     (cond ((not  iswitchb-matches)
-          (iswitchb-completion-help))
-         
+          (run-hooks 'iswitchb-cannot-complete-hook))
+
          ((= 1 (length iswitchb-matches))
           ;; only one choice, so select it.
           (exit-minibuffer))
-         
+
          (t
           ;; else there could be some completions
           (setq res iswitchb-common-match-string)
@@ -575,13 +675,13 @@ The result is stored in `iswitchb-common-match-string'."
                 (delete-region (minibuffer-prompt-end) (point))
                 (insert  res))
             ;; else nothing to complete
-            (iswitchb-completion-help)
+            (run-hooks 'iswitchb-cannot-complete-hook)
             )))))
 
 ;;; TOGGLE FUNCTIONS
 
 (defun iswitchb-toggle-case ()
-  "Toggle the value of `iswitchb-case'."
+  "Toggle the value of variable `iswitchb-case'."
   (interactive)
   (setq iswitchb-case (not iswitchb-case))
   ;; ask for list to be regenerated.
@@ -622,19 +722,19 @@ If no buffer exactly matching the prompt exists, maybe create a new one."
   (exit-minibuffer))
 
 (defun iswitchb-find-file ()
-  "Drop into find-file from buffer switching."
+  "Drop into `find-file' from buffer switching."
   (interactive)
   (setq iswitchb-exit 'findfile)
   (exit-minibuffer))
 
-(defun iswitchb-next-match () 
+(defun iswitchb-next-match ()
   "Put first element of `iswitchb-matches' at the end of the list."
   (interactive)
   (let ((next  (cadr iswitchb-matches)))
     (setq iswitchb-buflist (iswitchb-chop iswitchb-buflist next))
     (setq iswitchb-rescan t)))
 
-(defun iswitchb-prev-match () 
+(defun iswitchb-prev-match ()
   "Put last element of `iswitchb-matches' at the front of the list."
   (interactive)
   (let ((prev  (car (last iswitchb-matches))))
@@ -661,51 +761,53 @@ If no buffer exactly matching the prompt exists, maybe create a new one."
 (defun iswitchb-make-buflist (default)
   "Set `iswitchb-buflist' to the current list of buffers.
 Currently visible buffers are put at the end of the list.
-The hook `iswitchb-make-buflist-hook' is run after the list has been 
+The hook `iswitchb-make-buflist-hook' is run after the list has been
 created to allow the user to further modify the order of the buffer names
 in this list.  If DEFAULT is non-nil, and corresponds to an existing buffer,
 it is put to the start of the list."
-  (setq iswitchb-buflist 
+  (setq iswitchb-buflist
        (let* ((iswitchb-current-buffers (iswitchb-get-buffers-in-frames))
-             (iswitchb-temp-buflist
-              (delq nil 
-                    (mapcar
-                     (lambda (x)
-                       (let ((b-name (buffer-name x)))
-                         (if (not 
-                              (or 
-                               (iswitchb-ignore-buffername-p b-name)
-                               (memq b-name iswitchb-current-buffers)))
-                             b-name)))
-                     (buffer-list)))))
-         (nconc iswitchb-temp-buflist iswitchb-current-buffers)
+              (iswitchb-temp-buflist
+               (delq nil
+                     (mapcar
+                      (lambda (x)
+                        (let ((b-name (buffer-name x)))
+                          (if (not
+                               (or
+                                (iswitchb-ignore-buffername-p b-name)
+                                (memq b-name iswitchb-current-buffers)))
+                              b-name)))
+                      (buffer-list (and iswitchb-use-frame-buffer-list
+                                        (selected-frame)))))))
+         (setq iswitchb-temp-buflist
+               (nconc iswitchb-temp-buflist iswitchb-current-buffers))
          (run-hooks 'iswitchb-make-buflist-hook)
-         ;; Should this be after the hooks, or should the hooks be the
+        ;; Should this be after the hooks, or should the hooks be the
          ;; final thing to be run?
          (if default
              (progn
-               (setq iswitchb-temp-buflist 
+               (setq iswitchb-temp-buflist
                      (delete default iswitchb-temp-buflist))
-               (setq iswitchb-temp-buflist 
+               (setq iswitchb-temp-buflist
                      (cons default iswitchb-temp-buflist))))
          iswitchb-temp-buflist)))
 
 (defun iswitchb-to-end (lst)
   "Move the elements from LST to the end of `iswitchb-temp-buflist'."
-  (mapcar 
-   (lambda (elem)  
+  (mapcar
+   (lambda (elem)
      (setq iswitchb-temp-buflist (delq elem iswitchb-temp-buflist)))
    lst)
-  (nconc iswitchb-temp-buflist lst))
+  (setq iswitchb-temp-buflist (nconc iswitchb-temp-buflist lst)))
 
 (defun iswitchb-get-buffers-in-frames (&optional current)
   "Return the list of buffers that are visible in the current frame.
-If optional argument `current' is given, restrict searching to the
+If optional argument CURRENT is given, restrict searching to the
 current frame, rather than all frames, regardless of value of
 `iswitchb-all-frames'."
   (let ((iswitchb-bufs-in-frame nil))
     (walk-windows 'iswitchb-get-bufname nil
-                 (if current 
+                 (if current
                      nil
                    iswitchb-all-frames))
     iswitchb-bufs-in-frame))
@@ -735,7 +837,7 @@ current frame, rather than all frames, regardless of value of
   "Return buffers matching REGEXP.
 If STRING-FORMAT is nil, consider REGEXP as just a string.
 BUFFER-LIST can be list of buffers or list of strings."
-  (let* ((case-fold-search  iswitchb-case)
+  (let* ((case-fold-search  (iswitchb-case))
         ;; need reverse since we are building up list backwards
         (list              (reverse buffer-list))
          (do-string         (stringp (car list)))
@@ -743,16 +845,16 @@ BUFFER-LIST can be list of buffers or list of strings."
          ret)
     (mapcar
      (lambda (x)
-       
+
        (if do-string
           (setq name x)               ;We already have the name
         (setq name (buffer-name x)))
-       
+
        (cond
        ((and (or (and string-format (string-match regexp name))
                  (and (null string-format)
                       (string-match (regexp-quote regexp) name)))
-             
+
              (not (iswitchb-ignore-buffername-p name)))
         (setq ret (cons name ret))
           )))
@@ -787,7 +889,7 @@ BUFFER-LIST can be list of buffers or list of strings."
 (defun iswitchb-word-matching-substring (word)
   "Return part of WORD before 1st match to `iswitchb-change-word-sub'.
 If `iswitchb-change-word-sub' cannot be found in WORD, return nil."
-  (let ((case-fold-search iswitchb-case)) 
+  (let ((case-fold-search (iswitchb-case)))
     (let ((m (string-match iswitchb-change-word-sub word)))
       (if m
           (substring word m)
@@ -808,7 +910,7 @@ If `iswitchb-change-word-sub' cannot be found in WORD, return nil."
     (setq alist (mapcar 'iswitchb-makealist res)) ;; could use an  OBARRAY
 
     ;; try-completion returns t if there is an exact match.
-    (let ((completion-ignore-case iswitchb-case))
+    (let ((completion-ignore-case (iswitchb-case)))
 
     (try-completion subs alist))))
 
@@ -818,7 +920,7 @@ If `iswitchb-change-word-sub' cannot be found in WORD, return nil."
 
 ;; from Wayne Mesard <wmesard@esd.sgi.com>
 (defun iswitchb-rotate-list (lis)
-  "Destructively removes the last element from LIS.
+  "Destructively remove the last element from LIS.
 Return the modified list with the last element prepended to it."
   (if (<= (length lis) 1)
       lis
@@ -831,19 +933,17 @@ Return the modified list with the last element prepended to it."
       (cons (car las) lis))))
 
 (defun iswitchb-completion-help ()
-  "Show possible completions in a *Buffer Completions* buffer."
+  "Show possible completions in a *Completions* buffer."
   ;; we could allow this buffer to be used to select match, but I think
   ;; choose-completion-string will need redefining, so it just inserts
-  ;; choice with out any previous input.  
+  ;; choice with out any previous input.
   (interactive)
   (setq iswitchb-rescan nil)
-  (let ((completion-setup-hook nil)    ;disable fancy highlight/selection.
-       (buf (current-buffer))
-       (temp-buf "*Buffer Completions*")
-       (win)
-       (again (eq last-command this-command)))
+  (let ((buf (current-buffer))
+       (temp-buf "*Completions*")
+       (win))
 
-    (if again
+    (if (eq last-command this-command)
        ;; scroll buffer
        (progn
          (set-buffer temp-buf)
@@ -852,22 +952,24 @@ Return the modified list with the last element prepended to it."
              (set-window-start win (point-min))
            (scroll-other-window))
          (set-buffer buf))
-      
+
       (with-output-to-temp-buffer temp-buf
-       (if iswitchb-xemacs 
-           
+       (if (featurep 'xemacs)
+
            ;; XEmacs extents are put on by default, doesn't seem to be
            ;; any way of switching them off.
            (display-completion-list (if iswitchb-matches
                                         iswitchb-matches
                                       iswitchb-buflist)
                                     :help-string "iswitchb "
-                                  :activate-callback 
-                                  '(lambda (x y z) 
-                                     (message "doesn't work yet, sorry!")))
+                                    :activate-callback
+                                    (lambda (x y z)
+                                      (message "doesn't work yet, sorry!")))
          ;; else running Emacs
+         (with-current-buffer standard-output
+           (fundamental-mode))
          (display-completion-list (if iswitchb-matches
-                                    iswitchb-matches
+                                      iswitchb-matches
                                     iswitchb-buflist))
          )))))
 
@@ -890,7 +992,7 @@ Return the modified list with the last element prepended to it."
          ;; killed, so we can't rely on kill-buffer return value.
          (if (get-buffer buf)
              ;; buffer couldn't be killed.
-             (setq iswitchb-rescan t)  
+             (setq iswitchb-rescan t)
            ;; else buffer was killed so remove name from list.
            (setq iswitchb-buflist  (delq buf iswitchb-buflist)))))))
 
@@ -908,13 +1010,12 @@ Return the modified list with the last element prepended to it."
             (or (eq iswitchb-method 'always-frame)
                 (y-or-n-p "Jump to frame? ")))
        (setq newframe (window-frame win))
-       (raise-frame newframe)
-       (select-frame newframe)
-       (select-window win)
-       (if (not iswitchb-xemacs)
-           ;; reposition mouse to make frame active.  not needed in XEmacs
-           ;; This line came from the other-frame defun in Emacs.
-           (set-mouse-position (selected-frame) (1- (frame-width)) 0)))
+        (if (fboundp 'select-frame-set-input-focus)
+            (select-frame-set-input-focus newframe)
+          (raise-frame newframe)
+          (select-frame newframe)
+          )
+       (select-window win))
        (t
        ;;  No buffer in other frames...
        (switch-to-buffer buffer)
@@ -929,8 +1030,8 @@ Return the modified list with the last element prepended to it."
      ((eq iswitchb-method 'otherframe)
       (progn
        (switch-to-buffer-other-frame buffer)
-       (if (not iswitchb-xemacs)
-           (set-mouse-position (selected-frame) (1- (frame-width)) 0))
+       (if (fboundp 'select-frame-set-input-focus)
+            (select-frame-set-input-focus (selected-frame)))
        )))))
 
 (defun iswitchb-possible-new-buffer (buf)
@@ -940,7 +1041,7 @@ Return the modified list with the last element prepended to it."
     (if (and iswitchb-newbuffer
             (or
              (not iswitchb-prompt-newbuffer)
-             
+
              (and iswitchb-prompt-newbuffer
                   (y-or-n-p
                    (format
@@ -971,7 +1072,9 @@ If BUFFER is visible in the current frame, return nil."
 (defun iswitchb-default-keybindings ()
   "Set up default keybindings for `iswitchb-buffer'.
 Call this function to override the normal bindings.  This function also
-adds a hook to the minibuffer."
+adds a hook to the minibuffer.
+
+Obsolescent.  Use `iswitchb-mode'."
   (interactive)
   (add-hook 'minibuffer-setup-hook 'iswitchb-minibuffer-setup)
   (global-set-key "\C-xb" 'iswitchb-buffer)
@@ -1038,22 +1141,22 @@ This is a hack for XEmacs, and should really be handled by `iswitchb-exhibit'."
        (goto-char (point-min)))))
 
 ;; add this hook for XEmacs only.
-(if iswitchb-xemacs
-    (add-hook 'iswitchb-minibuffer-setup-hook 
+(if (featurep 'xemacs)
+    (add-hook 'iswitchb-minibuffer-setup-hook
              'iswitchb-init-XEmacs-trick))
 
 ;;; XEmacs / backspace key
 ;; For some reason, if the backspace key is pressed in XEmacs, the
 ;; line gets confused, so I've added a simple key definition to make
-;; backspace act like the normal delete key.  
+;; backspace act like the normal delete key.
 
 (defun iswitchb-xemacs-backspacekey ()
   "Bind backspace to `backward-delete-char'."
   (define-key iswitchb-mode-map '[backspace] 'backward-delete-char)
   (define-key iswitchb-mode-map '[(meta backspace)] 'backward-kill-word))
 
-(if iswitchb-xemacs
-    (add-hook 'iswitchb-define-mode-map-hook 
+(if (featurep 'xemacs)
+    (add-hook 'iswitchb-define-mode-map-hook
              'iswitchb-xemacs-backspacekey))
 
 ;;; ICOMPLETE TYPE CODE
@@ -1063,7 +1166,6 @@ This is a hack for XEmacs, and should really be handled by `iswitchb-exhibit'."
 Copied from `icomplete-exhibit' with two changes:
 1. It prints a default buffer name when there is no text yet entered.
 2. It calls my completion routine rather than the standard completion."
-
   (if iswitchb-use-mycompletion
       (let ((contents (buffer-substring (minibuffer-prompt-end) (point-max)))
            (buffer-undo-list t))
@@ -1083,19 +1185,14 @@ Copied from `icomplete-exhibit' with two changes:
          (iswitchb-set-common-completion)
 
          ;; Insert the match-status information:
-         (insert-string
-          (iswitchb-completions 
-           contents
-           minibuffer-completion-table
-           minibuffer-completion-predicate
-           (not minibuffer-completion-confirm)))
-         ))))
-
-(defun iswitchb-completions
-  (name candidates predicate require-match)
+         (insert (iswitchb-completions
+                  contents
+                  (not minibuffer-completion-confirm)))))))
+
+(defun iswitchb-completions (name require-match)
   "Return the string that is displayed after the user's text.
 Modified from `icomplete-completions'."
-  
+
   (let ((comps iswitchb-matches)
                                         ; "-determined" - only one candidate
         (open-bracket-determined (if require-match "(" "["))
@@ -1110,10 +1207,10 @@ Modified from `icomplete-completions'."
          (setq first (car comps))
          (setq first (format "%s" first))
          (put-text-property 0 (length first) 'face
-                            (if (= (length comps) 1) 
+                            (if (= (length comps) 1)
                                 'font-lock-comment-face
                               'font-lock-function-name-face)
-                            first) 
+                            first)
          (setq comps  (cons first (cdr comps)))))
 
     (cond ((null comps) (format " %sNo match%s"
@@ -1124,7 +1221,7 @@ Modified from `icomplete-completions'."
           (concat (if (and (> (length (car comps))
                               (length name)))
                       (concat open-bracket-determined
-                              ;; when there is one match, show the 
+                              ;; when there is one match, show the
                               ;; matching buffer name in full
                               (car comps)
                               close-bracket-determined)
@@ -1136,7 +1233,6 @@ Modified from `icomplete-completions'."
                  (most nil)
                  (most-len (length most))
                  most-is-exact
-                 first
                  (alternatives
                   (apply
                    (function concat)
@@ -1159,9 +1255,10 @@ Modified from `icomplete-completions'."
 
              ;; put in common completion item -- what you get by
              ;; pressing tab
-             (if (> (length iswitchb-common-match-string) (length name))
+             (if (and (stringp iswitchb-common-match-string)
+                      (> (length iswitchb-common-match-string) (length name)))
                  (concat open-bracket-determined
-                         (substring iswitchb-common-match-string 
+                         (substring iswitchb-common-match-string
                                     (length name))
                          close-bracket-determined))
              ;; end of partial matches...
@@ -1171,7 +1268,7 @@ Modified from `icomplete-completions'."
                   (concat open-bracket-determined
                           (substring most (length name))
                           close-bracket-determined))
-             
+
              ;; list all alternatives
              open-bracket-prospects
              (if most-is-exact
@@ -1182,21 +1279,11 @@ Modified from `icomplete-completions'."
 (defun iswitchb-minibuffer-setup ()
   "Set up minibuffer for `iswitchb-buffer'.
 Copied from `icomplete-minibuffer-setup-hook'."
-  (if (iswitchb-entryfn-p)
-      (progn
-
-       (make-local-variable 'iswitchb-use-mycompletion)
-       (setq iswitchb-use-mycompletion t)
-       (make-local-hook 'pre-command-hook)
-       (add-hook 'pre-command-hook
-                 'iswitchb-pre-command
-                 nil t)
-       (make-local-hook 'post-command-hook)
-       (add-hook 'post-command-hook
-                 'iswitchb-post-command
-                 nil t)
-       
-       (run-hooks 'iswitchb-minibuffer-setup-hook))))
+  (when (iswitchb-entryfn-p)
+    (set (make-local-variable 'iswitchb-use-mycompletion) t)
+    (add-hook 'pre-command-hook 'iswitchb-pre-command nil t)
+    (add-hook 'post-command-hook 'iswitchb-post-command nil t)
+    (run-hooks 'iswitchb-minibuffer-setup-hook)))
 
 (defun iswitchb-pre-command ()
   "Run before command in `iswitchb-buffer'."
@@ -1212,44 +1299,54 @@ Copied from `icomplete-tidy'."
 
   (if (and (boundp 'iswitchb-eoinput)
           iswitchb-eoinput)
-      
+
       (if (> iswitchb-eoinput (point-max))
          ;; Oops, got rug pulled out from under us - reinit:
          (setq iswitchb-eoinput (point-max))
        (let ((buffer-undo-list buffer-undo-list )) ; prevent entry
          (delete-region iswitchb-eoinput (point-max))))
-    
+
     ;; Reestablish the local variable 'cause minibuffer-setup is weird:
     (make-local-variable 'iswitchb-eoinput)
     (setq iswitchb-eoinput 1)))
 
 (defun iswitchb-entryfn-p ()
-  "Return non-nil if `this-command' shows we are using `iswitchb-buffer'."
-  (or (boundp 'iswitchb-prepost-hooks)
-      ;; I think the of this may be redundant, since the prepost hooks
-      ;; will always be set in the iswitchb defuns.
-      ;;(and (symbolp this-command)            ; ignore lambda functions
-      ;;(memq this-command
-      ;;        '(iswitchb-buffer
-      ;;          iswitchb-buffer-other-frame
-      ;;       iswitchb-display-buffer
-      ;;       iswitchb-buffer-other-window))))
-  ))
+  "Return non-nil if we are using `iswitchb-buffer'."
+  (eq iswitchb-minibuf-depth (minibuffer-depth)))
 
 (defun iswitchb-summaries-to-end ()
   "Move the summaries to the end of the list.
 This is an example function which can be hooked on to
 `iswitchb-make-buflist-hook'.  Any buffer matching the regexps
 `Summary' or `output\*$'are put to the end of the list."
-  (let ((summaries (delq nil (mapcar 
-                             (lambda (x) 
-                                (if (or 
-                                     (string-match "Summary" x)
-                                     (string-match "output\\*$" x))
-                                    x))
-                             iswitchb-temp-buflist))))
+  (let ((summaries (delq nil
+                        (mapcar
+                         (lambda (x)
+                           (if (string-match "Summary\\|output\\*$" x)
+                               x))
+                         iswitchb-temp-buflist))))
     (iswitchb-to-end summaries)))
 
+(defun iswitchb-case ()
+  "Return non-nil iff we should ignore case when matching.
+See the variable `iswitchb-case' for details."
+  (if iswitchb-case
+      (if (featurep 'xemacs)
+         (isearch-no-upper-case-p iswitchb-text)
+       (isearch-no-upper-case-p iswitchb-text t))))
+
+;;;###autoload
+(define-minor-mode iswitchb-mode
+  "Toggle Iswitchb global minor mode.
+With arg, turn Iswitchb mode on if and only iff ARG is positive.
+This mode enables switching between buffers using substrings.  See
+`iswitchb' for details."
+  nil nil iswitchb-global-map :global t :group 'iswitchb
+  (if iswitchb-mode
+      (add-hook 'minibuffer-setup-hook 'iswitchb-minibuffer-setup)
+    (remove-hook 'minibuffer-setup-hook 'iswitchb-minibuffer-setup)))
+
 (provide 'iswitchb)
 
+;;; arch-tag: d74198ae-753f-44f2-b34f-0c515398d90a
 ;;; iswitchb.el ends here