xemacs-beta: lisp/isearch-mode.el comparison

comparison lisp/isearch-mode.el @ 412:697ef44129c6 r21-2-14

Import from CVS: tag r21-2-14

author	cvs
date	Mon, 13 Aug 2007 11:20:41 +0200
parents	de805c49cfc1
children	11054d720c21

comparison

equal deleted inserted replaced

-:12e008d41344
+:697ef44129c6
 ;;; isearch-mode.el --- Incremental search minor mode.
-;; Copyright (C) 1992,93,94,95,96,97,98,1999 Free Software Foundation, Inc.
+;; Copyright (C) 1992, 1993, 1997 Free Software Foundation, Inc.
 ;; Author: Daniel LaLiberte <liberte@cs.uiuc.edu>
 ;; Maintainer: XEmacs Development Team
 ;; Keywords: extensions, dumped
 ;; WITHOUT ANY WARRANTY; without even the implied warranty of
 ;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
 ;; General Public License for more details.
 ;; You should have received a copy of the GNU General Public License
 ;; along with XEmacs; see the file COPYING.  If not, write to the
 ;; Free Software Foundation, 59 Temple Place - Suite 330,
 ;; Boston, MA 02111-1307, USA.
-;;; Synched up with: FSF 20.4.
+;;; Synched up with: Not synched with FSF.
 ;;; Commentary:
+;; LCD Archive Entry:
+;; isearch-mode|Daniel LaLiberte|liberte@cs.uiuc.edu
+;; |A minor mode replacement for isearch.el.
+;;====================================================================
 ;; Instructions
-;; Searching with isearch-mode.el should work just like isearch.el
+;; Searching with isearch-mode.el should work just like isearch.el,
-;; [the one from Emacs 18], except it is done in a temporary minor
+;; except it is done in a temporary minor mode that terminates when
-;; mode that terminates when you finish searching.
+;; you finish searching.
-;; For programmed use of isearch-mode, e.g. calling (isearch-forward),
+;; Semi-modal searching is supported, using a recursive edit. If
-;; isearch-mode behaves modally and does not return until the search
+;; isearching is started non-interactively by calling one of the
-;; is completed.  It uses a recursive-edit to behave this way.  In
+;; isearch commands (e.g. (isearch-forward), but not like gnus does
-;; that case, you should still be able switch buffers, so be careful
+;; it: (call-interactively 'isearch-forward)), isearch-mode does not
-;; not to get things confused.
+;; return until the search is completed.  You should still be able
+;; switch buffers, so be careful not to get things confused.
 ;; The key bindings active within isearch-mode are defined below in
 ;; `isearch-mode-map' which is given bindings close to the default
-;; characters of the original isearch.el.  With `isearch-mode',
+;; characters of isearch.el for version 19.  With `isearch-mode',
 ;; however, you can bind multi-character keys and it should be easier
 ;; to add new commands.  One bug though: keys with meta-prefix cannot
 ;; be longer than two chars.  Also see minibuffer-local-isearch-map
 ;; for bindings active during `isearch-edit-string'.
 ;; exits and does a nonincremental search.
 ;; Exiting immediately from isearch uses isearch-edit-string instead
 ;; of nonincremental-search, if search-nonincremental-instead is non-nil.
 ;; The name of this option should probably be changed if we decide to
-;; keep the behavior.  No point in forcing nonincremental search until
+;; keep the behavior.  One difference is that isearch-edit-string does
-;; the last possible moment.
+;; not support word search yet; perhaps isearch-mode should support it
+;; even for incremental searches, but how?
-;; TODO
-;; - Integrate generalized command history to isearch-edit-string.
+;;====================================================================
-;; - Think about incorporating query-replace.
+;;; Change History:
-;; - Hooks and options for failed search.
+;; Header: /import/kaplan/kaplan/liberte/Isearch/RCS/isearch-mode.el,v 1.3 92/06/29 13:10:08 liberte Exp Locker: liberte
-;;; Change Log:
+;; Log:	isearch-mode.el,v
+;;
-;; Changes before those recorded in ChangeLog:
 ;; 20-aug-92  Hacked by jwz for Lucid Emacs 19.3.
 ;;
 ;; Revision 1.3  92/06/29  13:10:08  liberte
 ;; Moved modal isearch-mode handling into isearch-mode.
 ;; Got rid of buffer-local isearch variables.
 ;; isearch-edit-string used by ring adjustments, completion, and
 ;; nonincremental searching.  C-s and C-r are additional exit commands.
 ;; Renamed all regex to regexp.
 ;; Got rid of found-start and found-point globals.
 ;; Generalized handling of upper-case chars.
 ;; Revision 1.2  92/05/27  11:33:57  liberte
 ;; Emacs version 19 has a search ring, which is supported here.
 ;; Other fixes found in the version 19 isearch are included here.
 ;;
 ;; Also see variables search-caps-disable-folding,
 ;; 3/18/92 Fixed invalid-regexp.
 ;; 3/18/92 Fixed yanking in regexps.
 ;;; Code:
+(defgroup isearch nil
+"Incremental search"
+:prefix "search-"
+:group 'matching)
+(defun isearch-char-to-string (c)
+(if (eventp c)
+(make-string 1 (event-to-character c nil nil t))
+(make-string 1 c)))
+;(defun isearch-text-char-description (c)
+;  (isearch-char-to-string c))
+(define-function 'isearch-text-char-description 'text-char-description)
 ;;;=========================================================================
 ;;; User-accessible variables
-(defgroup isearch nil
+(defvar search-last-string ""
-"Incremental search minor mode."
+"Last string search for by a search command.
-:prefix "search-"
+This does not include direct calls to the primitive search functions,
-:group 'matching)
+and does not include searches that are aborted.")
+(defvar search-last-regexp ""
-(defcustom search-exit-option t
+"Last string searched for by a regexp search command.
-"*Non-nil means random control characters terminate incremental search."
+This does not include direct calls to the primitive search functions,
-:type 'boolean
+and does not include searches that are aborted.")
-:group 'isearch)
+(defconst search-exit-option t
+"Non-nil means random control characters terminate incremental search.")
 (defcustom search-slow-window-lines 1
 "*Number of lines in slow search display windows.
 These are the short windows used during incremental search on slow terminals.
 Negative means put the slow search window at the top (normally it's at bottom)
 This is the style where a one-line window is created to show the line
 that the search has reached."
 :type 'integer
 :group 'isearch)
-;; We have `search-caps-disable-folding'.
-;(defcustom search-upper-case 'not-yanks
-;  "*If non-nil, upper case chars disable case fold searching.
-;That is, upper and lower case chars must match exactly.
-;This applies no matter where the chars come from, but does not
-;apply to chars in regexps that are prefixed with `\\'.
-;If this value is `not-yanks', yanked text is always downcased."
-;  :type '(choice (const :tag "off" nil)
-;		 (const not-yanks)
-;		 (other :tag "on" t))
-;  :group 'isearch)
 (defcustom search-nonincremental-instead t
-"*If non-nil, do a nonincremental search instead if exiting immediately.
+"*If non-nil, do a nonincremental search instead if exiting immediately."
-Actually, `isearch-edit-string' is called to let you enter the search
-string, and RET terminates editing and does a nonincremental search."
 :type 'boolean
 :group 'isearch)
-;; FSF default is "\\s-+", but I think our default is better so I'm
+(defcustom search-whitespace-regexp "\\(\\s \\|[\n\r]\\)+"
-;; leaving it.
-(defcustom search-whitespace-regexp "\\(\\s-\\|[\n\r]\\)+"
 "*If non-nil, regular expression to match a sequence of whitespace chars."
 :type 'regexp
 :group 'isearch)
-(defcustom search-highlight t
-"*Whether incremental search and query-replace should highlight
-the text that currently matches the search string."
-:type 'boolean
-:group 'isearch)
-;; I think the name `search-highlight' makes more sense, both because
-;; of consistency with other search-* variables above, and because it
-;; also applies to query-replace.
-(define-obsolete-variable-alias 'isearch-highlight 'search-highlight)
-(defcustom search-invisible 'open
-"If t incremental search can match hidden text.
-nil means don't match invisible text.
-If the value is `open', if the text matched is made invisible by
-an overlay having an `invisible' property and that overlay has a property
-`isearch-open-invisible', then incremental search will show the contents.
-\(This applies when using `outline.el' and `hideshow.el'.)"
-:type '(choice (const :tag "Match hidden text" t)
-		 (const :tag "Open overlays" open)
-		 (const :tag "Don't match hidden text" nil))
-:group 'isearch)
-(defcustom isearch-hide-immediately t
-"If non-nil, re-hide an invisible match right away.
-This variable makes a difference when `search-invisible' is set to `open'.
-It means that after search makes some invisible text visible
-to show the match, it makes the text invisible again when the match moves.
-Ordinarily the text becomes invisible again at the end of the search."
-:type 'boolean
-:group 'isearch)
-(defvar isearch-mode-hook nil
-"Function(s) to call after starting up an incremental search.")
-(defvar isearch-mode-end-hook nil
-"Function(s) to call after terminating an incremental search.")
 ;;;==================================================================
 ;;; Search ring.
 (defvar search-ring nil
 (defcustom regexp-search-ring-max 16
 "*Maximum length of regexp search ring before oldest elements are thrown away."
 :type 'integer
 :group 'isearch)
-;; The important difference between pre-20.4-merge yank-pointers and
-;; current code is that the yank pointers positions used to be
-;; preserved across the isearch sessions.  I changed this because I
-;; think the FSF code is closer to how the feature is supposed to
-;; behave (read: to minibuffer histories.)
 (defvar search-ring-yank-pointer nil
-"Index in `search-ring' of last string reused.
+"The tail of the search ring whose car is the last thing searched for.")
-nil if none yet.")
 (defvar regexp-search-ring-yank-pointer nil
-"Index in `regexp-search-ring' of last string reused.
+"The tail of the regular expression search ring whose car is the last
-nil if none yet.")
+thing searched for.")
-(defcustom search-ring-update nil
-"*Non-nil if advancing or retreating in the search ring should cause search.
-Default nil means edit the string from the search ring first."
-:type 'boolean
-:group 'isearch)
 ;;;====================================================
 ;;; Define isearch-mode keymap.
 (defvar isearch-mode-map
 (let ((map (make-keymap)))
 (set-keymap-name map 'isearch-mode-map)
 ;; Bind all printing characters to `isearch-printing-char'.
 ;; This isn't normally necessary, but if a printing character were
 ;; bound to something other than self-insert-command in global-map,
 ;; then it would terminate the search and be executed without this.
 (let ((i 32)
 	  (str (make-string 1 0)))
 (while (< i 127)
 	(aset str 0 i)
 	(define-key map str 'isearch-printing-char)
 	(setq i (1+ i))))
+(define-key map "\t" 'isearch-printing-char)
-;; Here FSF sets up various kludges to handle local bindings with
-;; meta char prefix keys.  We don't need isearch-other-meta-char
-;; because we handle things differently (via pre-command-hook).
 ;; Several non-printing chars change the searching behavior.
 ;;
 (define-key map "\C-s" 'isearch-repeat-forward)
 (define-key map "\M-\C-s" 'isearch-repeat-forward)
 (define-key map "\C-r" 'isearch-repeat-backward)
 (define-key map "\C-g" 'isearch-abort)
-;; I wish this worked...
-;(define-key map  [escape escape escape] 'isearch-cancel)
-(define-key map [(meta escape) escape] 'isearch-cancel)
 (define-key map "\C-q" 'isearch-quote-char)
 (define-key map "\C-m" 'isearch-exit)
 (define-key map "\C-j" 'isearch-printing-char)
 (define-key map "\t" 'isearch-printing-char)
-;; I prefer our default.
-;(define-key map " " 'isearch-whitespace-chars)
-(define-key map "\M- " 'isearch-whitespace-chars)
 (define-key map "\C-w" 'isearch-yank-word)
 (define-key map "\C-y" 'isearch-yank-line)
 (define-key map "\M-y" 'isearch-yank-kill)
-;; Define keys for regexp chars * ? |.
+;; Define keys for regexp chars * ? |
-;; Nothing special for + because it matches at least once.
 (define-key map "*" 'isearch-*-char)
 (define-key map "?" 'isearch-*-char)
 (define-key map "|" 'isearch-|-char)
+;; Some bindings you may want to put in your isearch-mode-hook.
+;; Suggest some alternates...
+;; (define-key map "\C-t" 'isearch-toggle-regexp)
+;; (define-key map "\C-^" 'isearch-edit-string)
 ;; delete and backspace delete backward, f1 is help, and C-h can be either
 (define-key map 'delete 'isearch-delete-char)
 (define-key map 'backspace 'isearch-delete-char)
 (define-key map '(control h) 'isearch-help-or-delete-char)
 (define-key map 'f1 'isearch-mode-help)
 (define-key map 'help 'isearch-mode-help)
 (define-key map "\M-n" 'isearch-ring-advance)
 (define-key map "\M-p" 'isearch-ring-retreat)
+(define-key map "\M- " 'isearch-whitespace-chars)
 (define-key map "\M-\t" 'isearch-complete)
-;; I find this binding somewhat unintuitive, because it doesn't
+(define-key map 'button2 'isearch-yank-x-selection)
-;; work if the mouse pointer is over the echo area -- it has to be
-;; over the search window.
-(define-key map 'button2 'isearch-yank-selection)
 map)
 "Keymap for isearch-mode.")
-;; Some bindings you may want to put in your isearch-mode-hook.
+(defvar minibuffer-local-isearch-map
-;; Suggest some alternates...
-;; (define-key isearch-mode-map "\C-t" 'isearch-toggle-case-fold)
-;; (define-key isearch-mode-map "\C-t" 'isearch-toggle-regexp)
-;; (define-key isearch-mode-map "\C-^" 'isearch-edit-string)
-(defvar minibuffer-local-isearch-map
 (let ((map (make-sparse-keymap)))
 ;; #### - this should also be minor-mode-ified
 (set-keymap-parents map (list minibuffer-local-map))
 (set-keymap-name map 'minibuffer-local-isearch-map)
 ;;#### This should just arrange to use the usual Emacs minibuffer histories
 (define-key map "\r" 'isearch-nonincremental-exit-minibuffer)
 (define-key map "\M-n" 'isearch-ring-advance-edit)
 (define-key map "\M-p" 'isearch-ring-retreat-edit)
-(define-key map 'down 'isearch-ring-advance-edit)
-(define-key map 'up 'isearch-ring-retreat-edit)
 (define-key map "\M-\t" 'isearch-complete-edit)
 (define-key map "\C-s" 'isearch-forward-exit-minibuffer)
 (define-key map "\C-r" 'isearch-reverse-exit-minibuffer)
 map)
 "Keymap for editing isearch strings in the minibuffer.")
 ;;;========================================================
 ;; Internal variables declared globally for byte-compiler.
-;; These are all set with setq while isearching
+;; These are all bound locally while editing the search string.
-;; and bound locally while editing the search string.
 (defvar isearch-forward nil)	; Searching in the forward direction.
 (defvar isearch-regexp nil)	; Searching for a regexp.
 (defvar isearch-word nil)	; Searching for words.
 (defvar isearch-string "")  ; The current search string.
 (defvar isearch-message "") ; text-char-description version of isearch-string
 (defvar isearch-success t)		; Searching is currently successful.
 (defvar isearch-invalid-regexp nil)	; Regexp not well formed.
-(defvar isearch-within-brackets nil)	; Regexp has unclosed [.
 (defvar isearch-other-end nil)	; Start (end) of match if forward (backward).
 (defvar isearch-wrapped nil)	; Searching restarted from the top (bottom).
 (defvar isearch-barrier 0)
 (defvar isearch-just-started nil)
 (defvar isearch-buffer nil)	; the buffer we've frobbed the keymap of
 (defvar isearch-case-fold-search nil)
-;; Need this for toggling case in isearch-toggle-case-fold.  When this
-;; is non-nil, the case-sensitiveness of the search is set by the
-;; user, and is may no longer be dynamically changed as per
-;; search-caps-disable-folding.
-(defvar isearch-fixed-case nil)
 (defvar isearch-adjusted nil)
 (defvar isearch-slow-terminal-mode nil)
 ;;; If t, using a small window.
 (defvar isearch-small-window nil)
 (defvar isearch-nonincremental nil)
 ;; New value of isearch-forward after isearch-edit-string.
 (defvar isearch-new-forward nil)
-;; Accumulate here the extents unhidden during searching.
-(defvar isearch-unhidden-extents nil)	; in FSF: isearch-opened-overlays
+(defvar isearch-mode-hook nil
+"Function(s) to call after starting up an incremental search.")
+(defvar isearch-mode-end-hook nil
+"Function(s) to call after terminating an incremental search.")
 ;;;==============================================================
 ;; Minor-mode-alist changes - kind of redundant with the
 ;; echo area, but if isearching in multiple windows, it can be useful.
 (add-minor-mode 'isearch-mode 'isearch-mode)
-(defvar isearch-mode nil) ;; Name of the minor mode, if non-nil.
+(defvar isearch-mode nil)
 (make-variable-buffer-local 'isearch-mode)
-;; We bind these in keydefs.el.
-;(define-key global-map "\C-s" 'isearch-forward)
-;(define-key global-map "\C-r" 'isearch-backward)
-;(define-key global-map "\M-\C-s" 'isearch-forward-regexp)
-;(define-key global-map "\M-\C-r" 'isearch-backward-regexp)
 ;;;===============================================================
 ;;; Entry points to isearch-mode.
 ;;; These four functions should replace those in loaddefs.el
-;;; An alternative is to defalias isearch-forward etc to isearch-mode,
+;;; An alternative is to fset isearch-forward etc to isearch-mode,
-;;; and look at this-command to set the options accordingly.
+;;; and look at the last command to set the options accordingly.
-(defun isearch-forward (&optional regexp-p no-recursive-edit)
+(defun isearch-forward (&optional regexp-p)
-"\
+"Do incremental search forward.
-Do incremental search forward.
 With a prefix argument, do an incremental regular expression search instead.
 \\<isearch-mode-map>
 As you type characters, they add to the search string and are found.
 The following non-printing keys are bound in `isearch-mode-map'.
 Type \\[isearch-delete-char] to cancel characters from end of search string.
 Type \\[isearch-exit] to exit, leaving point at location found.
 Type LFD (C-j) to match end of line.
 Type \\[isearch-repeat-forward] to search again forward,\
 \\[isearch-repeat-backward] to search again backward.
 Type \\[isearch-yank-word] to yank word from buffer onto end of search\
 string and search for it.
 Type \\[isearch-yank-line] to yank rest of line onto end of search string\
 and search for it.
-Type \\[isearch-yank-kill] to yank last killed text onto end of search string\
-and search for it.
 Type \\[isearch-quote-char] to quote control character to search for it.
 Type \\[isearch-whitespace-chars] to match all whitespace chars in regexp.
 \\[isearch-abort] while searching or when search has failed cancels input\
 back to what has
 been found successfully.
 ;; Non-standard bindings
 ;; Type \\[isearch-toggle-regexp] to toggle regular expression with normal searching.
 ;; Type \\[isearch-edit-string] to edit the search string in the minibuffer.
 ;;  Terminate editing and return to incremental searching with CR.
-(interactive "_P\np")
+(interactive "_P")
-(isearch-mode t (not (null regexp-p)) nil (not no-recursive-edit)))
+(isearch-mode t (not (null regexp-p)) nil (not (interactive-p))))
-(defun isearch-forward-regexp (&optional not-regexp no-recursive-edit)
+(defun isearch-forward-regexp ()
 "\
 Do incremental search forward for regular expression.
-With a prefix argument, do a regular string search instead.
 Like ordinary incremental search except that your input
 is treated as a regexp.  See \\[isearch-forward] for more info."
-(interactive "_P\np")
+(interactive "_")
-(isearch-mode t (null not-regexp) nil (not no-recursive-edit)))
+(isearch-mode t t nil (not (interactive-p))))
-(defun isearch-backward (&optional regexp-p no-recursive-edit)
+(defun isearch-backward (&optional regexp-p)
 "\
 Do incremental search backward.
-With a prefix argument, do a regular expression search instead.
+With a prefix argument, do an incremental regular expression search instead.
 See \\[isearch-forward] for more information."
-(interactive "_P\np")
+(interactive "_P")
-(isearch-mode nil (not (null regexp-p)) nil (not no-recursive-edit)))
+(isearch-mode nil (not (null regexp-p)) nil (not (interactive-p))))
-(defun isearch-backward-regexp (&optional not-regexp no-recursive-edit)
+(defun isearch-backward-regexp ()
 "\
 Do incremental search backward for regular expression.
-With a prefix argument, do a regular string search instead.
 Like ordinary incremental search except that your input
 is treated as a regexp.  See \\[isearch-forward] for more info."
-(interactive "_P\np")
+(interactive "_")
-(isearch-mode nil (null not-regexp) nil (not no-recursive-edit)))
+(isearch-mode nil t nil (not (interactive-p))))
-;; The problem here is that you can't scroll the help screen; as soon
+;; This function is way wrong, because you can't scroll the help
-;; as you press a key, it's gone.  I don't know of a good way to fix
+;; screen; as soon as you press a key, it's gone.  I don't know of a
-;; it, though.  -hniksic
+;; good way to fix it, though.  -hniksic
 (defun isearch-mode-help ()
 (interactive "_")
 (let ((w (selected-window)))
 (describe-function 'isearch-forward)
 (select-window w))
 ;;;==================================================================
 ;; isearch-mode only sets up incremental search for the minor mode.
 ;; All the work is done by the isearch-mode commands.
 (defun isearch-mode (forward &optional regexp op-fun recursive-edit word-p)
-"Start isearch minor mode.  Called by `isearch-forward', etc.
+"Start isearch minor mode.  Called by isearch-forward, etc."
-\\{isearch-mode-map}"
 (if executing-kbd-macro (setq recursive-edit nil))
 (let ((inhibit-quit t)) ; don't leave things in an inconsistent state...
 	  isearch-forward forward
 	  isearch-regexp regexp
 	  isearch-word word-p
 	  isearch-op-fun op-fun
 	  isearch-case-fold-search case-fold-search
-	  isearch-fixed-case nil
 	  isearch-string ""
 	  isearch-message ""
 	  isearch-cmds nil
 	  isearch-success t
 	  isearch-wrapped nil
 	  isearch-barrier (point)
 	  isearch-adjusted nil
 	  isearch-yank-flag nil
 	  isearch-invalid-regexp nil
-	  isearch-within-brackets nil
 	  isearch-slow-terminal-mode (and (<= (device-baud-rate)
 					      search-slow-speed)
 					  (> (window-height)
 					     (* 4 search-slow-window-lines)))
 	  isearch-other-end nil
 	  isearch-small-window nil
 	  isearch-just-started t
 	  isearch-opoint (point)
-	  search-ring-yank-pointer nil
-	  regexp-search-ring-yank-pointer nil
-	  isearch-unhidden-extents nil
 	  isearch-window-configuration (current-window-configuration)
-	  ;; #### What we really need is a buffer-local
+	  ;; #### Should we remember the old value of
-	  ;; overriding-local-map.  See isearch-pre-command-hook for
+	  ;; overriding-local-map?
-	  ;; more details.
 	  overriding-local-map (progn
 				 (set-keymap-parents isearch-mode-map
 				  (nconc (current-minor-mode-maps)
 					 (and (current-local-map)
 					      (list (current-local-map)))))
 				 isearch-mode-map)
 	  isearch-selected-frame (selected-frame)
+	  isearch-mode (gettext " Isearch")
 	  )
 ;; XEmacs change: without clearing the match data, sometimes old values
 ;; of isearch-other-end get used.  Don't ask me why...
 (store-match-data nil)
 (add-hook 'pre-command-hook 'isearch-pre-command-hook)
+(set-buffer-modified-p (buffer-modified-p)) ; update modeline
-(setq isearch-mode (gettext " Isearch"))
-(redraw-modeline)
 (isearch-push-state)
 ) ; inhibit-quit is t before here
 (isearch-update)
 (run-hooks 'isearch-mode-hook)
 ;; isearch-mode can be made modal (in the sense of not returning to
 ;; the calling function until searching is completed) by entering
 ;; a recursive-edit and exiting it when done isearching.
 (if recursive-edit
 (let ((isearch-recursive-edit t))
 	(recursive-edit)))
-isearch-success)
+)
 ;;;====================================================
 ;; Some high level utilities.  Others below.
 (defun isearch-update ()
 ;; Called after each command to update the display.
-(if (null unread-command-events)
+(if (null unread-command-event)
 (progn
 	(if (not (input-pending-p))
 	    (isearch-message))
 	(if (and isearch-slow-terminal-mode
 		 (not (or isearch-small-window
 			  (pos-visible-in-window-p))))
 	    (let ((found-point (point)))
 	      (setq isearch-small-window t)
 	      (move-to-window-line 0)
 	      (let ((window-min-height 1))
 	      (goto-char found-point)))
 	(if isearch-other-end
 	    (if (< isearch-other-end (point))
 		(isearch-highlight isearch-other-end (point))
 	      (isearch-highlight (point) isearch-other-end))
-	  (isearch-dehighlight))
+	  (if (extentp isearch-extent)
+	      (isearch-dehighlight nil)))
 	))
 (setq ;; quit-flag nil  not for isearch-mode
 isearch-adjusted nil
 isearch-yank-flag nil)
-(isearch-highlight-all-update)
 )
-(defun isearch-done (&optional nopush edit)
+(defun isearch-done ()
 ;; Called by all commands that terminate isearch-mode.
 (let ((inhibit-quit t)) ; danger danger!
 (if (and isearch-buffer (buffer-live-p isearch-buffer))
-	;; Some loser process filter might have switched the window's
+	(save-excursion
-	;; buffer, so be sure to set these variables back in the
+	  ;; Some loser process filter might have switched the
-	;; buffer we frobbed them in.  But only if the buffer is still
+	  ;; window's buffer, so be sure to set these variables back
-	;; alive.
+	  ;; in the buffer we frobbed them in.  But only if the buffer
-	(with-current-buffer isearch-buffer
+	  ;; is still alive.
+	  (set-buffer isearch-buffer)
+	  ;; #### Should we restore the old value of
+	  ;; overriding-local-map?
 	  (setq overriding-local-map nil)
 	  ;; Use remove-hook instead of just setting it to our saved value
 	  ;; in case some process filter has created a buffer and modified
 	  ;; the pre-command-hook in that buffer...  yeah, this is obscure,
 	  ;; and yeah, I was getting screwed by it. -jwz
 	  (remove-hook 'pre-command-hook 'isearch-pre-command-hook)
 	  (set-keymap-parents isearch-mode-map nil)
 	  (setq isearch-mode nil)
-	  (redraw-modeline)
+	  (set-buffer-modified-p (buffer-modified-p));; update modeline
-	  (isearch-dehighlight)
+	  (isearch-dehighlight t)))
-	  (isearch-highlight-all-cleanup)
-	  (isearch-restore-invisible-extents nil nil)
-	  ))
 ;; it's not critical that this be inside inhibit-quit, but leaving
 ;; things in small-window-mode would be bad.
 (let ((found-start (window-start (selected-window)))
 	  (found-point (point)))
 	       (set-window-start (selected-window) found-start t))))
 ;; If there was movement, mark the starting position.
 ;; Maybe should test difference between and set mark iff > threshold.
 (if (and (buffer-live-p isearch-buffer)
 	       (/= (point isearch-buffer) isearch-opoint))
-	  ;; #### FSF doesn't do this if the region is active.  Should
-	  ;; we do the same?
 	  (progn
 	    (push-mark isearch-opoint t nil isearch-buffer)
 	    (or executing-kbd-macro (> (minibuffer-depth) 0)
-		(display-message 'command "Mark saved where search started")))))
+		(display-message 'command "Mark saved where search started"))))
+	)
 (setq isearch-buffer nil)
 ) ; inhibit-quit is t before here
-(if (and (> (length isearch-string) 0) (not nopush))
+(if (> (length isearch-string) 0)
 ;; Update the ring data.
-(isearch-update-ring isearch-string isearch-regexp))
+(if isearch-regexp
+	  (if (not (setq regexp-search-ring-yank-pointer
+			 (member isearch-string regexp-search-ring)))
+	      (progn
+		(setq regexp-search-ring
+		      (cons isearch-string regexp-search-ring)
+		      regexp-search-ring-yank-pointer regexp-search-ring)
+		(if (> (length regexp-search-ring) regexp-search-ring-max)
+		    (setcdr (nthcdr (1- regexp-search-ring-max) regexp-search-ring)
+			    nil))))
+	(if (not (setq search-ring-yank-pointer
+		       ;; really need equal test instead of eq.
+		       (member isearch-string search-ring)))
+	    (progn
+	      (setq search-ring (cons isearch-string search-ring)
+		    search-ring-yank-pointer search-ring)
+	      (if (> (length search-ring) search-ring-max)
+		  (setcdr (nthcdr (1- search-ring-max) search-ring) nil))))))
 (run-hooks 'isearch-mode-end-hook)
+(if isearch-recursive-edit (exit-recursive-edit)))
-(and (not edit) isearch-recursive-edit (exit-recursive-edit)))
-(defun isearch-update-ring (string &optional regexp)
-"Add STRING to the beginning of the search ring.
-REGEXP says which ring to use."
-(if regexp
-(if (or (null regexp-search-ring)
-	      (not (string= string (car regexp-search-ring))))
-	  (progn
-	    (setq regexp-search-ring
-		  (cons string regexp-search-ring))
-	    (if (> (length regexp-search-ring) regexp-search-ring-max)
-		(setcdr (nthcdr (1- search-ring-max) regexp-search-ring)
-			nil))))
-(if (or (null search-ring)
-	    (not (string= string (car search-ring))))
-	(progn
-	  (setq search-ring (cons string search-ring))
-	  (if (> (length search-ring) search-ring-max)
-	      (setcdr (nthcdr (1- search-ring-max) search-ring) nil))))))
 ;;;====================================================
 ;; Commands active while inside of the isearch minor mode.
 (defun isearch-exit ()
 "Exit search normally.
 However, if this is the first command after starting incremental
-search and `search-nonincremental-instead' is non-nil, do a
+search and `search-nonincremental-instead' is non-nil, do an
-nonincremental search instead via `isearch-edit-string'."
+incremental search via `isearch-edit-string'."
 (interactive)
 (if (and search-nonincremental-instead
 	   (= 0 (length isearch-string)))
-(let ((isearch-nonincremental t)
+(let ((isearch-nonincremental t))
-	    ;; Highlighting only gets in the way of nonincremental
-	    ;; search.
-	    (search-highlight nil)
-	    (isearch-highlight-all-matches nil))
 	(isearch-edit-string))
 (isearch-done)))
 (defun isearch-edit-string ()
 "Edit the search string in the minibuffer.
 The following additional command keys are active while editing.
 \\<minibuffer-local-isearch-map>
-\\[exit-minibuffer] to resume incremental searching with the edited string.
+\\[exit-minibuffer] to exit editing and resume incremental searching.
-\\[isearch-nonincremental-exit-minibuffer] to do one nonincremental search.
 \\[isearch-forward-exit-minibuffer] to resume isearching forward.
-\\[isearch-reverse-exit-minibuffer] to resume isearching backward.
+\\[isearch-backward-exit-minibuffer] to resume isearching backward.
-\\[isearch-ring-advance-edit] to replace the search string with the next item in the search ring.
+\\[isearch-ring-advance-edit] to replace the search string with the next\
-\\[isearch-ring-retreat-edit] to replace the search string with the previous item in the search ring.
+item in the search ring.
-\\[isearch-complete-edit] to complete the search string using the search ring.
+\\[isearch-ring-retreat-edit] to replace the search string with the next\
-\\<isearch-mode-map>
+item in the search ring.
-If first char entered is \\[isearch-yank-word], then do word search instead."
+\\[isearch-complete-edit] to complete the search string from the search ring."
-;; This code is very hairy for several reasons, explained in the code.
-;; Mainly, isearch-mode must be terminated while editing and then restarted.
-;; If there were a way to catch any change of buffer from the minibuffer,
-;; this could be simplified greatly.
 ;; Editing doesn't back up the search point.  Should it?
 (interactive)
 (condition-case nil
-(progn
+(let ((minibuffer-local-map minibuffer-local-isearch-map)
-	(let ((isearch-nonincremental isearch-nonincremental)
+	    isearch-nonincremental	; should search nonincrementally?
+	    isearch-new-string
-	      ;; Locally bind all isearch global variables to protect them
+	    isearch-new-message
-	      ;; from recursive isearching.
+	    (isearch-new-forward isearch-forward)
-	      ;; isearch-string -message and -forward are not bound
-	      ;; so they may be changed.  Instead, save the values.
+	    ;; Locally bind all isearch global variables to protect them
-	      (isearch-new-string isearch-string)
+	    ;; from recursive isearching.
-	      (isearch-new-message isearch-message)
+	    (isearch-string isearch-string)
-	      (isearch-new-forward isearch-forward)
+	    (isearch-message isearch-message)
-	      (isearch-new-word isearch-word)
+	    (isearch-forward isearch-forward) ; set by commands below.
-	      (isearch-regexp isearch-regexp)
+	    (isearch-forward isearch-forward)
-	      (isearch-op-fun isearch-op-fun)
+	    (isearch-regexp isearch-regexp)
-	      (isearch-cmds isearch-cmds)
+	    (isearch-word isearch-word)
-	      (isearch-success isearch-success)
+	    (isearch-op-fun isearch-op-fun)
-	      (isearch-wrapped isearch-wrapped)
+	    (isearch-cmds isearch-cmds)
-	      (isearch-barrier isearch-barrier)
+	    (isearch-success isearch-success)
-	      (isearch-adjusted isearch-adjusted)
+	    (isearch-wrapped isearch-wrapped)
-	      (isearch-fixed-case isearch-fixed-case)
+	    (isearch-barrier isearch-barrier)
-	      (isearch-yank-flag isearch-yank-flag)
+	    (isearch-adjusted isearch-adjusted)
-	      (isearch-invalid-regexp isearch-invalid-regexp)
+	    (isearch-yank-flag isearch-yank-flag)
-	      (isearch-within-brackets isearch-within-brackets)
+	    (isearch-invalid-regexp isearch-invalid-regexp)
-;;; Don't bind this.  We want isearch-search, below, to set it.
+	    (isearch-other-end isearch-other-end)
-;;; And the old value won't matter after that.
+	    (isearch-opoint isearch-opoint)
-;;;	    (isearch-other-end isearch-other-end)
+	    (isearch-slow-terminal-mode isearch-slow-terminal-mode)
-	      (isearch-opoint isearch-opoint)
+	    (isearch-small-window isearch-small-window)
-	      (isearch-slow-terminal-mode isearch-slow-terminal-mode)
+	    (isearch-recursive-edit isearch-recursive-edit)
-	      (isearch-small-window isearch-small-window)
+	    (isearch-window-configuration (current-window-configuration))
-	      (isearch-recursive-edit isearch-recursive-edit)
+	    (isearch-selected-frame (selected-frame))
-	      (isearch-window-configuration (current-window-configuration))
+	    )
-	      (isearch-selected-frame (selected-frame))
+	;; Actually terminate isearching until editing is done.
+	;; This is so that the user can do anything without failure,
+	;; like switch buffers and start another isearch, and return.
+;;	(condition-case nil
+	    (isearch-done)
+;;#### What does this mean?  There is no such condition!
+;;	  (exit nil))			; was recursive editing
+	(unwind-protect
+	    (let ((prompt (isearch-message-prefix nil t))
+event)
+	      ;; If the first character the user types when we prompt them
+	      ;; for a string is the yank-word character, then go into
+	      ;; word-search mode.  Otherwise unread that character and
+	      ;; read a string the normal way.
+	      (let ((cursor-in-echo-area t))
+		(display-message 'prompt prompt)
+		(setq event (next-command-event))
+		(if (eq 'isearch-yank-word
+			(lookup-key isearch-mode-map (vector event)))
+		    (setq isearch-word t)
+		  (setq unread-command-event event)))
+	      (setq isearch-new-string
+;;                    (if (fboundp 'gmhist-old-read-from-minibuffer)
+;;                        ;; Eschew gmhist crockery
+;;			(gmhist-old-read-from-minibuffer prompt isearch-string)
+		      (read-string
+		       prompt isearch-string
+		       't            ;does its own history (but shouldn't)
+;;                     (if isearch-regexp
+;;                         ;; The search-rings aren't exactly minibuffer
+;;                         ;;  histories, but they are close enough
+;;                         (cons 'regexp-search-ring
+;;                               (- (length regexp-search-ring-yank-pointer)
+;;                                  (length regexp-search-ring)))
+;;                         (cons 'search-ring
+;;                               (- (length search-ring-yank-pointer)
+;;                                  (length search-ring))))
+		       )
+;;		      )
+		    isearch-new-message (mapconcat
+					 'isearch-text-char-description
+					 isearch-new-string ""))
 	      )
-	  ;; Actually terminate isearching until editing is done.
+	  ;; Always resume isearching by restarting it.
-	  ;; This is so that the user can do anything without failure,
+	  (isearch-mode isearch-forward
-	  ;; like switch buffers and start another isearch, and return.
+			isearch-regexp
-;;	(condition-case nil
+			isearch-op-fun
-	  (isearch-done t t)
+			isearch-recursive-edit
-;;#### What does this mean?  There is no such condition!
+			isearch-word)
-;;	(exit nil))			; was recursive editing
+	  )
-	  (unwind-protect
+	;; Copy new values in outer locals to isearch globals
-	      (progn
+	(setq isearch-string isearch-new-string
-		;; Fake the prompt message for the sake of
+	      isearch-message isearch-new-message
-		;; next-command-event below.
+	      isearch-forward isearch-new-forward)
-		(isearch-message)
-		;; If the first character the user types when we
+	;; Empty isearch-string means use default.
-		;; prompt them for a string is the yank-word
+	(if (= 0 (length isearch-string))
-		;; character, then go into word-search mode.
+	    (setq isearch-string (if isearch-regexp search-last-regexp
-		;; Otherwise unread that character and read a string
+				   search-last-string))
-		;; the normal way.
+	  ;; Set last search string now so it is set even if we fail.
-		(let* ((cursor-in-echo-area t)
+	  (if search-last-regexp
-		       (event (next-command-event)))
+	      (setq search-last-regexp isearch-string)
-		  (if (eq 'isearch-yank-word
+	    (setq search-last-string isearch-string)))
-			  (lookup-key isearch-mode-map (vector event)))
-		      (setq isearch-word t;; so message-prefix is right
-			    isearch-new-word t)
-		    (setq unread-command-event event)))
-		(setq isearch-new-string
-		      (read-from-minibuffer
-		       (isearch-message-prefix nil isearch-nonincremental)
-		       isearch-string
-		       minibuffer-local-isearch-map
-		       nil
-		       't		;does its own history (but shouldn't)
-		       )
-		      isearch-new-message (mapconcat
-					   'isearch-text-char-description
-					   isearch-new-string "")))
-	    ;; Always resume isearching by restarting it.
-	    (isearch-mode isearch-forward
-			  isearch-regexp
-			  isearch-op-fun
-			  isearch-recursive-edit
-			  isearch-word)
-	    ;; Copy new values in outer locals to isearch globals
-	    (setq isearch-string isearch-new-string
-		  isearch-message isearch-new-message
-		  isearch-forward isearch-new-forward
-		  isearch-word isearch-new-word))
-	  ;; Empty isearch-string means use default.
-	  (if (= 0 (length isearch-string))
-	      (setq isearch-string (or (car (if isearch-regexp
-						regexp-search-ring
-					      search-ring))
-				       ""))))
 	;; Reinvoke the pending search.
 	(isearch-push-state)
 	(isearch-search)
 	(isearch-update)
 (defun isearch-reverse-exit-minibuffer ()
 (interactive)
 (setq isearch-new-forward nil)
 (exit-minibuffer))
-(defun isearch-cancel ()
-"Terminate the search and go back to the starting point."
-(interactive)
-(goto-char isearch-opoint)
-(isearch-done t)
-(signal 'quit '(isearch)))		; and pass on quit signal
 (defun isearch-abort ()
-"Abort incremental search mode if searching is successful, signaling quit.
+"Quit incremental search mode if searching is successful, signalling quit.
 Otherwise, revert to previous successful search and continue searching.
-Use `isearch-exit' to quit without signaling."
+Use `isearch-exit' to quit without signalling."
 (interactive)
-;;  (ding)  signal instead below, if quitting
+;;  (ding)  signal instead below, if quiting
 (discard-input)
 (if isearch-success
 ;; If search is successful, move back to starting point
 ;; and really do quit.
 (progn (goto-char isearch-opoint)
-	     (setq isearch-success nil)
+	     (isearch-done)   ; exit isearch
-	     (isearch-done t)   ; exit isearch
 	     (signal 'quit '(isearch)))  ; and pass on quit signal
-;; If search is failing, or has an incomplete regexp,
+;; If search is failing, rub out until it is once more successful.
-;; rub out until it is once more successful.
+(while (not isearch-success) (isearch-pop-state))
-(while (or (not isearch-success) isearch-invalid-regexp)
-(isearch-pop-state))
 (isearch-update)))
 (defun isearch-repeat (direction)
 ;; Utility for isearch-repeat-forward and -backward.
 (if (eq isearch-forward (eq direction 'forward))
 ;; C-s in forward or C-r in reverse.
 (if (equal isearch-string "")
 	  ;; If search string is empty, use last one.
 	  (setq isearch-string
 		(or (if isearch-regexp
-			(car regexp-search-ring)
+			(if regexp-search-ring-yank-pointer
-		      (car search-ring))
+			    (car regexp-search-ring-yank-pointer)
+			  (car regexp-search-ring))
+		      (if search-ring-yank-pointer
+			  (car search-ring-yank-pointer)
+			(car search-ring)))
 		    "")
 		isearch-message
 		(mapconcat 'isearch-text-char-description
 			   isearch-string ""))
 	;; If already have what to search for, repeat it.
 	(or isearch-success
 	    (progn
 	      (goto-char (if isearch-forward (point-min) (point-max)))
 	      (setq isearch-wrapped t))))
 ;; C-s in reverse or C-r in forward, change direction.
 (setq isearch-forward (not isearch-forward)))
 (setq isearch-barrier (point)) ; For subsequent \| if regexp.
 (if (equal isearch-string "")
 (setq isearch-success t)
-(if (and isearch-success (equal (match-end 0) (match-beginning 0))
+(if (and (equal (match-end 0) (match-beginning 0))
+	     isearch-success
 	     (not isearch-just-started))
 	;; If repeating a search that found
 	;; an empty string, ensure we advance.
 	(if (if isearch-forward (eobp) (bobp))
-	    ;; If there's nowhere to advance to, fail (and wrap next time).
+	    ;; nowhere to advance to, so fail (and wrap next time)
 	    (progn
 	      (setq isearch-success nil)
 	      (and executing-kbd-macro
 		   (not defining-kbd-macro)
 		   (isearch-done))
 	      (ding nil 'isearch-failed))
 	  (forward-char (if isearch-forward 1 -1))
 	  (isearch-search))
 (isearch-search)))
 (isearch-push-state)
 (isearch-update))
 (defun isearch-repeat-forward ()
 "Repeat incremental search forwards."
 (isearch-update))
 (defun isearch-toggle-case-fold ()
 "Toggle case folding in searching on or off."
 (interactive)
-(setq isearch-case-fold-search (if isearch-case-fold-search nil 'yes)
+(setq isearch-case-fold-search
-	isearch-fixed-case t)
+	(if isearch-case-fold-search nil 'yes))
-(lmessage 'progress "%s%s [case %ssensitive]"
+(message "%s%s [case %ssensitive]"
-(isearch-message-prefix)
+	   (isearch-message-prefix)
-isearch-message
+	   isearch-message
-(if isearch-case-fold-search "in" ""))
+	   (if isearch-case-fold-search "in" ""))
 (setq isearch-adjusted t)
-;; Update the highlighting here so that it gets done before the
-;; one-second pause.
-(isearch-highlight-all-update)
 (sit-for 1)
 (isearch-update))
 (defun isearch-delete-char ()
 "Discard last input item and move point back.
 If no previous match was done, just beep."
 (interactive)
 (if (null (cdr isearch-cmds))
 (ding nil 'isearch-quit)
 (isearch-pop-state))
 ('tty (eq tty-erase-char ?\C-h))
 ('x (not (x-keysym-on-keyboard-sans-modifiers-p 'backspace)))))
 (isearch-delete-char)
 (isearch-mode-help)))
-;; This is similar to FSF isearch-yank-string, but more general.
 (defun isearch-yank (chunk)
 ;; Helper for isearch-yank-* functions.  CHUNK can be a string or a
 ;; function.
 (let ((word (if (stringp chunk)
 		  chunk
 		(save-excursion
 		  (and (not isearch-forward) isearch-other-end
 		       (goto-char isearch-other-end))
 		  (buffer-substring
 		   (point)
-		   (progn
+		   (save-excursion
 		     (funcall chunk)
 		     (point)))))))
 ;; if configured so that typing upper-case characters turns off case
 ;; folding, then downcase the string so that yanking an upper-case
 ;; word doesn't mess with case-foldedness.
 			     word ""))
 	  ;; Don't move cursor in reverse search.
 	  isearch-yank-flag t))
 (isearch-search-and-update))
 (defun isearch-yank-word ()
 "Pull next word from buffer into search string."
 (interactive)
 (isearch-yank (function (lambda () (forward-word 1)))))
 (defun isearch-yank-sexp ()
 "Pull next expression from buffer into search string."
 (interactive)
 (isearch-yank 'forward-sexp))
-(defun isearch-yank-selection ()
+(defun isearch-yank-x-selection ()
-"Pull the current selection into the search string."
+"Pull the current X selection into the search string."
 (interactive)
-(isearch-yank (get-selection)))
+(isearch-yank (x-get-selection)))
-(defun isearch-yank-clipboard ()
+(defun isearch-yank-x-clipboard ()
-"Pull the current clipboard selection into the search string."
+"Pull the current X clipboard selection into the search string."
 (interactive)
-(isearch-yank (get-clipboard)))
+(isearch-yank (x-get-clipboard)))
 (defun isearch-fix-case ()
-;; The commented-out (and ...) form implies that, once
+(if (and isearch-case-fold-search search-caps-disable-folding)
-;; isearch-case-fold-search becomes nil due to a capital letter
+(setq isearch-case-fold-search
-;; typed in, it can never be restored to the original value.  In
-;; that case, it's impossible to revert a case-sensitive search back
-;; to case-insensitive.
-(if ;(and isearch-case-fold-search search-caps-disable-folding)
-(and case-fold-search
-	   ;; Make sure isearch-toggle-case-fold works.
-	   (not isearch-fixed-case)
-	   search-caps-disable-folding)
-(setq isearch-case-fold-search
 	    (no-upper-case-p isearch-string isearch-regexp)))
 (setq isearch-mode (if case-fold-search
 (if isearch-case-fold-search
 " Isearch"  ;As God Intended Mode
 			   " ISeARch") ;Warn about evil case via StuDLYcAps.
-		       " Isearch")))
+		       "Isearch"
+;		         (if isearch-case-fold-search
+;                            " isearch"    ;Presumably case-sensitive losers
+;                                          ;will notice this 1-char difference.
+;                            " Isearch")   ;Weenie mode.
+			 )))
 (defun isearch-search-and-update ()
 ;; Do the search and update the display.
 (if (and (not isearch-success)
 	   ;; unsuccessful regexp search may become
 		   (let ((case-fold-search isearch-case-fold-search))
 		     (looking-at (if isearch-regexp isearch-string
 				   (regexp-quote isearch-string)))))
 	       (error nil))
 	     (or isearch-yank-flag
 		 (<= (match-end 0)
 		     (min isearch-opoint isearch-barrier))))
 	(setq isearch-success t
 	      isearch-invalid-regexp nil
-	      isearch-within-brackets nil
 	      isearch-other-end (match-end 0))
 ;; Not regexp, not reverse, or no match at point.
 (if (and isearch-other-end (not isearch-adjusted))
 	  (goto-char (if isearch-forward isearch-other-end
 		       (min isearch-opoint
 			    isearch-barrier
 			    (1+ isearch-other-end)))))
 (isearch-search)
 ))
 (isearch-push-state)
 (if isearch-op-fun (funcall isearch-op-fun))
 (isearch-update))
 ;; *, ?, and | chars can make a regexp more liberal.
-;; They can make a regexp match sooner or make it succeed instead of failing.
+;; They can make a regexp match sooner
+;; or make it succeed instead of failing.
 ;; So go back to place last successful search started
 ;; or to the last ^S/^R (barrier), whichever is nearer.
-;; + needs no special handling because the string must match at least once.
 (defun isearch-*-char ()
 "Handle * and ? specially in regexps."
 (interactive)
 (if isearch-regexp
-(let ((idx (length isearch-string)))
-	(while (and (> idx 0)
+(progn
-		    (eq (aref isearch-string (1- idx)) ?\\))
+	(setq isearch-adjusted t)
-	  (setq idx (1- idx)))
+	(let ((cs (nth (if isearch-forward
-	(when (= (mod (- (length isearch-string) idx) 2) 0)
+			   5		; isearch-other-end
-	  (setq isearch-adjusted t)
+			 2)		; saved (point)
-	  ;; Get the isearch-other-end from before the last search.
+		       (car (cdr isearch-cmds)))))
-	  ;; We want to start from there,
-	  ;; so that we don't retreat farther than that.
 	  ;; (car isearch-cmds) is after last search;
 	  ;; (car (cdr isearch-cmds)) is from before it.
-	  (let ((cs (nth 5 (car (cdr isearch-cmds)))))
+	  (setq cs (or cs isearch-barrier))
-	    (setq cs (or cs isearch-barrier))
+	  (goto-char
-	    (goto-char
+	   (if isearch-forward
-	     (if isearch-forward
+	       (max cs isearch-barrier)
-		 (max cs isearch-barrier)
+	     (min cs isearch-barrier))))))
-	       (min cs isearch-barrier)))))))
 (isearch-process-search-char last-command-event))
 (defun isearch-|-char ()
 "If in regexp search, jump to the barrier."
 (interactive)
 (progn
 	(setq isearch-adjusted t)
 	(goto-char isearch-barrier)))
 (isearch-process-search-char last-command-event))
-;; FSF:
-;(defalias 'isearch-other-control-char 'isearch-other-meta-char)
-;
-;(defun isearch-other-meta-char ()
-;...
-;
 (defun isearch-quote-char ()
 "Quote special characters for incremental search."
 (interactive)
-;; #### Here FSF does some special conversion of chars in 0200-0377
-;; range.  Maybe we should do the same.
 (isearch-process-search-char (read-quoted-char (isearch-message t))))
 (defun isearch-return-char ()
 "Convert return into newline for incremental search.
 Obsolete."
 (interactive)
 (isearch-process-search-char ?\n))
 (defun isearch-printing-char ()
-"Add this ordinary printing character to the search string and search."
+"Any other printing character => add it to the search string and search."
 (interactive)
-(let ((event last-command-event))
+(isearch-process-search-char last-command-event))
-;; If we are called by isearch-whitespace-chars because the
-;; context disallows whitespace search (e.g. within brackets),
-;; replace M-SPC with a space.  FSF has similar code.
-(and (eq this-command 'isearch-whitespace-chars)
-	 (null (event-to-character event))
-	 (setq event (character-to-event ?\ )))
-(isearch-process-search-char event)))
 (defun isearch-whitespace-chars ()
 "Match all whitespace chars, if in regexp mode."
-;; FSF docstring adds: "If you want to search for just a space, type
+(interactive)
-;; C-q SPC."  But we don't need the addition because we have a
+(if (and isearch-regexp search-whitespace-regexp)
-;; different (better) default for the variable.
+(isearch-process-search-string search-whitespace-regexp " ")
-(interactive)
+(beep)
-(if isearch-regexp
+(isearch-process-search-char ?\ )
-(if (and search-whitespace-regexp (not isearch-within-brackets)
+;    (if isearch-word
-	       (not isearch-invalid-regexp))
+;	nil
-	  (isearch-process-search-string search-whitespace-regexp " ")
+;      (setq isearch-word t)
-	(isearch-printing-char))
+;      (goto-char isearch-other-end)
-(progn
+;      (isearch-process-search-char ?\ ))
-;; This way of doing word search doesn't correctly extend current search.
+))
-;;      (setq isearch-word t)
-;;      (setq isearch-adjusted t)
-;;      (goto-char isearch-barrier)
-(isearch-printing-char))))
 (defun isearch-process-search-char (char)
 ;; Append the char to the search string, update the message and re-search.
 (isearch-process-search-string (isearch-char-to-string char)
 				 (isearch-text-char-description char)))
 (defun isearch-process-search-string (string message)
 (setq isearch-string (concat isearch-string string)
 	isearch-message (concat isearch-message message))
 ;;===========================================================
 ;; Search Ring
+(defcustom search-ring-update nil
+"*Non-nil if advancing or retreating in the search ring should cause search.
+Default nil means edit the string from the search ring first."
+:type 'boolean
+:group 'isearch)
 (defun isearch-ring-adjust1 (advance)
 ;; Helper for isearch-ring-adjust
 (let* ((ring (if isearch-regexp regexp-search-ring search-ring))
 	 (length (length ring))
 	 (yank-pointer-name (if isearch-regexp
 	 (yank-pointer (eval yank-pointer-name)))
 (if (zerop length)
 	()
 (set yank-pointer-name
 	   (setq yank-pointer
-		 (mod (+ (or yank-pointer 0)
+		 (nthcdr (% (+ (- length (length yank-pointer))
-			 (if advance -1 1))
+			       (if advance (1- length) 1))
-		      length)))
+			    length) ring)))
-(setq isearch-string (nth yank-pointer ring)
+(setq isearch-string (car yank-pointer)
 	    isearch-message (mapconcat 'isearch-text-char-description
 				       isearch-string "")))))
 (defun isearch-ring-adjust (advance)
 ;; Helper for isearch-ring-advance and isearch-ring-retreat
-;  (if (cdr isearch-cmds)  ;; is there more than one thing on stack?
+(if (cdr isearch-cmds)  ;; is there more than one thing on stack?
-;      (isearch-pop-state))
+(isearch-pop-state))
 (isearch-ring-adjust1 advance)
+(isearch-push-state)
 (if search-ring-update
 (progn
 	(isearch-search)
 	(isearch-update))
 (isearch-edit-string)
-)
+))
-(isearch-push-state))
 (defun isearch-ring-advance ()
 "Advance to the next search string in the ring."
 ;; This could be more general to handle a prefix arg, but who would use it.
 (interactive)
 (defun isearch-ring-retreat ()
 "Retreat to the previous search string in the ring."
 (interactive)
 (isearch-ring-adjust nil))
-(defun isearch-ring-advance-edit (n)
+(defun isearch-ring-adjust-edit (advance)
-"Insert the next element of the search history into the minibuffer."
+"Use the next or previous search string in the ring while in minibuffer."
-(interactive "p")
+(isearch-ring-adjust1 advance)
-(let* ((yank-pointer-name (if isearch-regexp
+(erase-buffer)
-				'regexp-search-ring-yank-pointer
+(insert isearch-string))
-			      'search-ring-yank-pointer))
-	 (yank-pointer (eval yank-pointer-name))
+(defun isearch-ring-advance-edit ()
-	 (ring (if isearch-regexp regexp-search-ring search-ring))
+(interactive)
-	 (length (length ring)))
+(isearch-ring-adjust-edit 'advance))
-(if (zerop length)
-	()
+(defun isearch-ring-retreat-edit ()
-(set yank-pointer-name
+"Retreat to the previous search string in the ring while in the minibuffer."
-	   (setq yank-pointer
+(interactive)
-		 (mod (- (or yank-pointer 0) n)
+(isearch-ring-adjust-edit nil))
-		      length)))
-(erase-buffer)
-(insert (nth yank-pointer ring))
-(goto-char (point-max)))))
-(defun isearch-ring-retreat-edit (n)
-"Inserts the previous element of the search history into the minibuffer."
-(interactive "p")
-(isearch-ring-advance-edit (- n)))
-;; Merging note: FSF comments out these functions and implements them
-;; differently (see above), presumably because the versions below mess
-;; with isearch-string, while what we really want them to do is simply
-;; to insert the correct string to the minibuffer.
-;;(defun isearch-ring-adjust-edit (advance)
-;;  "Use the next or previous search string in the ring while in minibuffer."
-;;  (isearch-ring-adjust1 advance)
-;;  (erase-buffer)
-;;  (insert isearch-string))
-;;(defun isearch-ring-advance-edit ()
-;;  (interactive)
-;;  (isearch-ring-adjust-edit 'advance))
-;;(defun isearch-ring-retreat-edit ()
-;;  "Retreat to the previous search string in the ring while in the minibuffer."
-;;  (interactive)
-;;  (isearch-ring-adjust-edit nil))
 (defun isearch-complete1 ()
 ;; Helper for isearch-complete and isearch-complete-edit
-;; Return t if completion OK, nil if no completion exists.
+;; Return t if completion OK,
 (let* ((ring (if isearch-regexp regexp-search-ring search-ring))
 (alist (mapcar (function (lambda (string) (list string))) ring))
 (completion-ignore-case case-fold-search)
-(completion (try-completion isearch-string alist)))
+(completion (try-completion isearch-string alist))
+	 )
 (cond
 ((eq completion t)
 ;; isearch-string stays the same
 t)
 ((or completion ; not nil, must be a string
 	  (= 0 (length isearch-string))) ; shouldn't have to say this
 (if (equal completion isearch-string)  ;; no extension?
-	  (progn
+	  (if completion-auto-help
-	    (if completion-auto-help
+	      (with-output-to-temp-buffer "*Isearch completions*"
-		(with-output-to-temp-buffer "*Isearch completions*"
+		(display-completion-list
-		  (display-completion-list
+		 (all-completions isearch-string alist))))
-		   (all-completions isearch-string alist))))
+	(setq isearch-string completion))
-	    t)
+t)
-	(and completion
-	     (setq isearch-string completion))))
 (t
 (temp-minibuffer-message "No completion")
 nil))))
 (defun isearch-complete ()
 	(erase-buffer)
 	(insert isearch-string))))
 ;;;==============================================================
-;; The search status stack.
+;; The search status stack (and isearch window-local variables, not used).
 (defun isearch-top-state ()
+;;  (fetch-window-local-variables)
 (let ((cmd (car isearch-cmds)))
-;; #### Grr, this is so error-prone.  If you add something to
-;; isearch-push-state, don't forget to update this.  I thought I'd
-;; make a list of variables, and just do (mapcar* #'set vars
-;; values), but the (point) thing would spoil it, leaving to more
-;; complication.
 (setq isearch-string (car cmd)
 	  isearch-message (car (cdr cmd))
 	  isearch-success (nth 3 cmd)
 	  isearch-forward (nth 4 cmd)
 	  isearch-other-end (nth 5 cmd)
-	  isearch-word (nth 6 cmd)
+	  isearch-invalid-regexp (nth 6 cmd)
-	  isearch-invalid-regexp (nth 7 cmd)
+	  isearch-wrapped (nth 7 cmd)
-	  isearch-wrapped (nth 8 cmd)
+	  isearch-barrier (nth 8 cmd))
-	  isearch-barrier (nth 9 cmd)
-	  isearch-within-brackets (nth 10 cmd))
 (goto-char (car (cdr (cdr cmd))))))
 (defun isearch-pop-state ()
-(pop isearch-cmds)
+;;  (fetch-window-local-variables)
+(setq isearch-cmds (cdr isearch-cmds))
 (isearch-top-state)
+)
-;; Make sure isearch-case-fold-search gets the correct value.  FSF
-;; simply stores isearch-case-fold-search to isearch-cmds.  We
-;; should probably do the same.
-(isearch-fix-case)
-;; Here, as well as in isearch-search we must deal with the point
-;; landing at an invisible area which may need unhiding.
-(if (or (not (eq search-invisible 'open))
-	  (not isearch-hide-immediately))
-;; If search-invisible is t, invisible text is just like any
-;; other text.  If it is nil, it is always skipped and we can't
-;; land inside.  In both cases, we don't need to do anything.
-;;
-;; Similarly, if isearch-hide-immediately is nil, needn't
-;; re-hide the area here, and neither can we land back into a
-;; hidden one.
-nil
-(when isearch-other-end
-;; This will unhide the extents.
-(isearch-range-invisible (point) isearch-other-end))
-(isearch-restore-invisible-extents (point)
-				       (or isearch-other-end (point)))))
 (defun isearch-push-state ()
 (setq isearch-cmds
 	(cons (list isearch-string isearch-message (point)
 		    isearch-success isearch-forward isearch-other-end
-		    isearch-word
+		    isearch-invalid-regexp isearch-wrapped isearch-barrier)
-		    isearch-invalid-regexp isearch-wrapped isearch-barrier
-		    isearch-within-brackets)
 	      isearch-cmds)))
 ;;;==================================================================
 ;; Message string
 (defun isearch-message (&optional c-q-hack ellipsis)
 ;; Generate and print the message string.
 (let ((cursor-in-echo-area ellipsis)
 	(m (concat
-	    (isearch-message-prefix c-q-hack ellipsis isearch-nonincremental)
+	    (isearch-message-prefix c-q-hack)
 	    isearch-message
-	    (isearch-message-suffix c-q-hack ellipsis)
+	    (isearch-message-suffix c-q-hack)
 	    )))
-(if c-q-hack
+(if c-q-hack m (display-message 'progress (format "%s" m)))))
-	m
-(display-message 'progress (format "%s" m)))))
+(defun isearch-message-prefix (&optional c-q-hack nonincremental)
-(defun isearch-message-prefix (&optional c-q-hack ellipsis nonincremental)
 ;; If about to search, and previous search regexp was invalid,
 ;; check that it still is.  If it is valid now,
 ;; let the message we display while searching say that it is valid.
-(and isearch-invalid-regexp ellipsis
+(and isearch-invalid-regexp
 (condition-case ()
 	   (progn (re-search-forward isearch-string (point) t)
-		  (setq isearch-invalid-regexp nil
+		  (setq isearch-invalid-regexp nil))
-			isearch-within-brackets nil))
 	 (error nil)))
-;; If currently failing, display no ellipsis.
+;; #### - Yo!  Emacs assembles strings all over the place, they can't all
-(or isearch-success (setq ellipsis nil))
+;; be internationalized in the manner proposed below...  Add an explicit
-;; #### - !  Emacs assembles strings all over the place, they can't
+;; call to `gettext' and have the string snarfer pluck the english
-;; all be internationalized in the manner proposed below...  Add an
+;; strings out of the comment below.  XEmacs is on a purespace diet! -Stig
-;; explicit call to `gettext' and have the string snarfer pluck the
-;; english strings out of the comment below.  XEmacs is on a
-;; purespace diet! -Stig
-;; The comment below is dead and buried, but it can be rebuilt if
-;; necessary.  -hniksic
 (let ((m (concat (if isearch-success nil "failing ")
-		   (if (and isearch-wrapped
+		   (if isearch-wrapped "wrapped ")
-			    (if isearch-forward
-				(> (point) isearch-opoint)
-			      (< (point) isearch-opoint)))
-		       "overwrapped "
-		     (if isearch-wrapped "wrapped "))
 		   (if isearch-word "word ")
 		   (if isearch-regexp "regexp ")
 		   (if nonincremental "search" "I-search")
 		   (if isearch-forward nil " backward")
 		   ": "
 		   )))
 (aset m 0 (upcase (aref m 0)))
 (gettext m)))
-(defun isearch-message-suffix (&optional c-q-hack ellipsis)
+(defun isearch-message-suffix (&optional c-q-hack)
 (concat (if c-q-hack "^Q" "")
 	  (if isearch-invalid-regexp
 	      (concat " [" isearch-invalid-regexp "]")
 	    "")))
+;;;;; #### - yuck...this is soooo lame.  Is this really worth 4k of purespace???
+;;;
 ;;;(let ((i (logior (if isearch-success 32 0)
 ;;;                (if isearch-wrapped 16 0)
 ;;;                (if isearch-word     8 0)
 ;;;                (if isearch-regexp   4 0)
 ;;;                (if nonincremental   2 0)
 ;;;                (if isearch-forward  1 0))))
 ;;;  (cond
 ;;;   ((= i 63) (gettext "Wrapped word regexp search: "))              ; 111111
-;;;   ...and so on, ad nauseam...
+;;;   ((= i 62) (gettext "Wrapped word regexp search backward: "))     ; 111110
+;;;   ((= i 61) (gettext "Wrapped word regexp I-search: "))            ; 111101
+;;;   ((= i 60) (gettext "Wrapped word regexp I-search backward: "))   ; 111100
+;;;   ((= i 59) (gettext "Wrapped word search: "))                     ; 111011
+;;;   ((= i 58) (gettext "Wrapped word search backward: "))            ; 111010
+;;;   ((= i 57) (gettext "Wrapped word I-search: "))                   ; 111001
+;;;   ((= i 56) (gettext "Wrapped word I-search backward: "))          ; 111000
+;;;   ((= i 55) (gettext "Wrapped regexp search: "))                   ; 110111
+;;;   ((= i 54) (gettext "Wrapped regexp search backward: "))          ; 110110
+;;;   ((= i 53) (gettext "Wrapped regexp I-search: "))                 ; 110101
+;;;   ((= i 52) (gettext "Wrapped regexp I-search backward: "))        ; 110100
+;;;   ((= i 51) (gettext "Wrapped search: "))                          ; 110011
+;;;   ((= i 50) (gettext "Wrapped search backward: "))                 ; 110010
+;;;   ((= i 49) (gettext "Wrapped I-search: "))                        ; 110001
+;;;   ((= i 48) (gettext "Wrapped I-search backward: "))	       ; 110000
+;;;   ((= i 47) (gettext "Word regexp search: "))                      ; 101111
+;;;   ((= i 46) (gettext "Word regexp search backward: "))             ; 101110
+;;;   ((= i 45) (gettext "Word regexp I-search: "))                    ; 101101
+;;;   ((= i 44) (gettext "Word regexp I-search backward: "))           ; 101100
+;;;   ((= i 43) (gettext "Word search: "))                             ; 101011
+;;;   ((= i 42) (gettext "Word search backward: "))                    ; 101010
+;;;   ((= i 41) (gettext "Word I-search: "))                           ; 101001
+;;;   ((= i 40) (gettext "Word I-search backward: "))                  ; 101000
+;;;   ((= i 39) (gettext "Regexp search: "))                           ; 100111
+;;;   ((= i 38) (gettext "Regexp search backward: "))                  ; 100110
+;;;   ((= i 37) (gettext "Regexp I-search: "))                         ; 100101
+;;;   ((= i 36) (gettext "Regexp I-search backward: "))                ; 100100
+;;;   ((= i 35) (gettext "Search: "))                                  ; 100011
+;;;   ((= i 34) (gettext "Search backward: "))                         ; 100010
+;;;   ((= i 33) (gettext "I-search: "))                                ; 100001
+;;;   ((= i 32) (gettext "I-search backward: "))		       ; 100000
+;;;   ((= i 31) (gettext "Failing wrapped word regexp search: "))      ; 011111
+;;;   ((= i 30) (gettext "Failing wrapped word regexp search backward: ")) ; 011110
+;;;   ((= i 29) (gettext "Failing wrapped word regexp I-search: "))    ; 011101
+;;;   ((= i 28) (gettext "Failing wrapped word regexp I-search backward: ")) ; 011100
+;;;   ((= i 27) (gettext "Failing wrapped word search: "))             ; 011011
+;;;   ((= i 26) (gettext "Failing wrapped word search backward: "))    ; 011010
+;;;   ((= i 25) (gettext "Failing wrapped word I-search: "))           ; 011001
+;;;   ((= i 24) (gettext "Failing wrapped word I-search backward: "))  ; 011000
+;;;   ((= i 23) (gettext "Failing wrapped regexp search: "))           ; 010111
+;;;   ((= i 22) (gettext "Failing wrapped regexp search backward: "))  ; 010110
+;;;   ((= i 21) (gettext "Failing wrapped regexp I-search: "))         ; 010101
+;;;   ((= i 20) (gettext "Failing wrapped regexp I-search backward: ")) ; 010100
+;;;   ((= i 19) (gettext "Failing wrapped search: "))                  ; 010011
+;;;   ((= i 18) (gettext "Failing wrapped search backward: "))         ; 010010
+;;;   ((= i 17) (gettext "Failing wrapped I-search: "))                ; 010001
+;;;   ((= i 16) (gettext "Failing wrapped I-search backward: "))       ; 010000
+;;;   ((= i 15) (gettext "Failing word regexp search: "))              ; 001111
+;;;   ((= i 14) (gettext "Failing word regexp search backward: "))     ; 001110
+;;;   ((= i 13) (gettext "Failing word regexp I-search: "))            ; 001101
+;;;   ((= i 12) (gettext "Failing word regexp I-search backward: "))   ; 001100
+;;;   ((= i 11) (gettext "Failing word search: "))                     ; 001011
+;;;   ((= i 10) (gettext "Failing word search backward: "))            ; 001010
+;;;   ((= i  9) (gettext "Failing word I-search: "))                   ; 001001
+;;;   ((= i  8) (gettext "Failing word I-search backward: "))          ; 001000
+;;;   ((= i  7) (gettext "Failing regexp search: "))                   ; 000111
+;;;   ((= i  6) (gettext "Failing regexp search backward: "))          ; 000110
+;;;   ((= i  5) (gettext "Failing regexp I-search: "))                 ; 000101
+;;;   ((= i  4) (gettext "Failing regexp I-search backward: "))        ; 000100
+;;;   ((= i  3) (gettext "Failing search: "))                          ; 000011
+;;;   ((= i  2) (gettext "Failing search backward: "))                 ; 000010
+;;;   ((= i  1) (gettext "Failing I-search: "))                        ; 000001
 ;;;   ((= i  0) (gettext "Failing I-search backward: "))	       ; 000000
 ;;;   (t (error "Something's rotten")))))
 ;;;========================================================
 (put 'isearch-return-char			'isearch-command t)
 (put 'isearch-repeat-forward			'isearch-command t)
 (put 'isearch-repeat-backward			'isearch-command t)
 (put 'isearch-delete-char			'isearch-command t)
 (put 'isearch-help-or-delete-char		'isearch-command t)
-(put 'isearch-cancel				'isearch-command t)
 (put 'isearch-abort				'isearch-command t)
 (put 'isearch-quote-char			'isearch-command t)
 (put 'isearch-exit				'isearch-command t)
 (put 'isearch-printing-char			'isearch-command t)
 (put 'isearch-printing-char			'isearch-command t)
 (put 'isearch-edit-string			'isearch-command t)
 (put 'isearch-toggle-regexp			'isearch-command t)
 (put 'isearch-forward-exit-minibuffer		'isearch-command t)
 (put 'isearch-reverse-exit-minibuffer		'isearch-command t)
 (put 'isearch-nonincremental-exit-minibuffer	'isearch-command t)
-(put 'isearch-yank-selection			'isearch-command t)
-(put 'isearch-yank-clipboard			'isearch-command t)
 (put 'isearch-yank-x-selection			'isearch-command t)
 (put 'isearch-yank-x-clipboard			'isearch-command t)
 ;; scrolling the scrollbar should not terminate isearch.
 ;; them in the buffer.  In this way, the set of self-searching characters
 ;; need not be exhaustively enumerated, but is derived from other maps.
 ;;
 (cond ((not (eq (current-buffer) isearch-buffer))
 	 ;; If the buffer (likely meaning "frame") has changed, bail.
-	 ;; This can happen if the user types something into another
+	 ;; This can also happen if a proc filter has popped up another
-	 ;; frame.  It can also happen if a proc filter has popped up
+	 ;; buffer, which is arguably a bad thing for it to have done,
-	 ;; another buffer, which is arguably a bad thing for it to
+	 ;; but the way in which isearch would have hosed you in that
-	 ;; have done, but the way in which isearch would have hosed
+	 ;; case is unarguably even worse. -jwz
-	 ;; you in that case is unarguably even worse. -jwz
+	 (isearch-done))
-	 (isearch-done)
-	 ;; `this-command' is set according to the value of
-	 ;; `overriding-local-map', set by isearch-mode.  This is
-	 ;; wrong because that keymap makes sense only in isearch
-	 ;; buffer.  To make sure the right command is called, adjust
-	 ;; `this-command' to the appropriate value, now that
-	 ;; `isearch-done' has set `overriding-local-map' to nil.
-	 ;; FSF does similar magic in `isearch-other-meta-char', which
-	 ;; is horribly complex.  I *hope* what we do works in all
-	 ;; cases.
-	 (setq this-command (key-binding (this-command-keys))))
 	(t
 	 (isearch-maybe-frob-keyboard-macros)
 	 (if (and this-command
 		  (symbolp this-command)
 		  (get this-command 'isearch-command))
 ;;;========================================================
 ;;; Highlighting
+(defcustom isearch-highlight t
+"*Whether isearch and query-replace should highlight the text which
+currently matches the search-string.")
 (defvar isearch-extent nil)
-;; this face is initialized by faces.el since isearch is preloaded.
+;; this face is initialized by x-faces.el since isearch is preloaded.
-;(make-face 'isearch)
+;; this face is now created in initialize-faces
+;;(make-face 'isearch)
 (defun isearch-make-extent (begin end)
 (let ((x (make-extent begin end (current-buffer))))
-;; make the isearch extent always take precedence over any mouse-
+;; make the isearch extent always take prescedence over any mouse-
 ;; highlighted extents we may be passing through, since isearch, being
 ;; modal, is more interesting (there's nothing they could do with a
 ;; mouse-highlighted extent while in the midst of a search anyway).
-(set-extent-priority x (+ mouse-highlight-priority 2))
+(set-extent-priority x (1+ mouse-highlight-priority))
 (set-extent-face x 'isearch)
 (setq isearch-extent x)))
 (defun isearch-highlight (begin end)
-(if (null search-highlight)
+(if (null isearch-highlight)
 nil
 ;; make sure isearch-extent is in the current buffer
-(or (and (extentp isearch-extent)
+(or (extentp isearch-extent)
-	     (extent-live-p isearch-extent))
 	(isearch-make-extent begin end))
 (set-extent-endpoints isearch-extent begin end (current-buffer))))
-;; This used to have a TOTALLY flag that also deleted the extent.  I
+(defun isearch-dehighlight (totally)
-;; don't think this is necessary any longer, as isearch-highlight can
+(if (and isearch-highlight isearch-extent)
-;; simply move the extent to another buffer.  The IGNORED argument is
+(if totally
-;; for the code that calls this function with an argument.  --hniksic
+	  (let ((inhibit-quit t))
-(defun isearch-dehighlight (&optional ignored)
+	    (if (extentp isearch-extent)
-(and search-highlight
+		(delete-extent isearch-extent))
-(extentp isearch-extent)
+	    (setq isearch-extent nil))
-(extent-live-p isearch-extent)
+	(if (extentp isearch-extent)
-(detach-extent isearch-extent)))
+	    (detach-extent isearch-extent)
+	  (setq isearch-extent nil)))))
 ;;;========================================================
 ;;; Searching
 ;; Do the search with the current search string.
 (isearch-message nil t)
 (isearch-fix-case)
 (condition-case lossage
 (let ((inhibit-quit nil)
-	    (case-fold-search isearch-case-fold-search)
+	    (case-fold-search isearch-case-fold-search))
-	    (retry t))
 	(if isearch-regexp (setq isearch-invalid-regexp nil))
-	(setq isearch-within-brackets nil)
+	(setq isearch-success
-	(while retry
+	      (funcall
-	  (setq isearch-success
+	       (cond (isearch-word
-		(funcall
+		      (if isearch-forward
-		 (cond (isearch-word
+			  'word-search-forward 'word-search-backward))
-			(if isearch-forward
+		     (isearch-regexp
-			    'word-search-forward 'word-search-backward))
+		      (if isearch-forward
-		       (isearch-regexp
+			  're-search-forward 're-search-backward))
-			(if isearch-forward
+		     (t
-			    're-search-forward 're-search-backward))
+		      (if isearch-forward 'search-forward 'search-backward)))
-		       (t
+	       isearch-string nil t))
-			(if isearch-forward 'search-forward 'search-backward)))
-		 isearch-string nil t))
-	  ;; Clear RETRY unless we matched some invisible text
-	  ;; and we aren't supposed to do that.
-	  (if (or (eq search-invisible t)
-		  (not isearch-success)
-		  (bobp) (eobp)
-		  (= (match-beginning 0) (match-end 0))
-		  (not (isearch-range-invisible
-			(match-beginning 0) (match-end 0))))
-	      (setq retry nil)))
 	(setq isearch-just-started nil)
-	(when isearch-success
+	(if isearch-success
-	  (setq isearch-other-end
+	    (setq isearch-other-end
-		(if isearch-forward (match-beginning 0) (match-end 0)))
+		  (if isearch-forward (match-beginning 0) (match-end 0)))))
-	  (and isearch-hide-immediately
-	       (isearch-restore-invisible-extents (match-beginning 0)
+(quit (setq unread-command-event (character-to-event (quit-char)))
-						  (match-end 0)))))
-(quit (setq unread-command-events (nconc unread-command-events
-					     (character-to-event (quit-char))))
 	  (setq isearch-success nil))
 (invalid-regexp
 (setq isearch-invalid-regexp (car (cdr lossage)))
-(setq isearch-within-brackets (string-match "\\`Unmatched \\["
-						 isearch-invalid-regexp))
 (if (string-match
 	  "\\`Premature \\|\\`Unmatched \\|\\`Invalid "
 	  isearch-invalid-regexp)
-	 (setq isearch-invalid-regexp (gettext "incomplete input"))))
+	 (setq isearch-invalid-regexp (gettext "incomplete input")))))
-(error
-;; stack overflow in regexp search.
-(setq isearch-invalid-regexp (car (cdr lossage)))))
 (if isearch-success
 nil
 ;; If we're being run inside a keyboard macro, then the call to
 ;; Ding if failed this time after succeeding last time.
 (and (nth 3 (car isearch-cmds))
 	 (ding nil 'isearch-failed))
 (goto-char (nth 2 (car isearch-cmds)))))
-;; Replaced with isearch-edit-string.
+;;;=================================================
+;; This is called from incremental-search
+;; if the first input character is the exit character.
+;; We store the search string in `isearch-string'
+;; which has been bound already by `isearch-search'
+;; so that, when we exit, it is copied into `search-last-string'.
 ;(defun nonincremental-search (forward regexp)
-;...
+;  ;; This may be broken.  Anyway, it is replaced by the isearch-edit-string.
+;  ;; Missing features: word search option, command history.
-(defun isearch-unhide-extent (extent)
+;  (setq isearch-forward forward
-;; Store the values for the `invisible' and `intangible'
+;	isearch-regexp regexp)
-;; properties, and then set them to nil. This way the text hidden
+;  (let (char function
-;; by this extent becomes visible.
+;	inhibit-quit
-(put extent 'isearch-invisible (get extent 'invisible))
+;	(cursor-in-echo-area t))
-(put extent 'isearch-intangible (get extent 'intangible))
+;    ;; Prompt assuming not word search,
-(put extent 'invisible nil)
+;    (setq isearch-message
-(put extent 'intangible nil))
+;	  (if isearch-regexp
+;	      (if isearch-forward "Regexp search: "
-(defun isearch-range-invisible (beg end)
+;		"Regexp search backward: ")
-"Return t if all the text from BEG to END is invisible.
+;	    (if isearch-forward "Search: " "Search backward: ")))
-Before that, if search-invisible is `open', unhide the extents with an
+;    (message "%s" isearch-message)
-`isearch-open-invisible' property."
+;    ;; Read 1 char and switch to word search if it is ^W.
-;; isearch-search uses this to skip the extents that are invisible,
+;    (setq char (read-char))
-;; but don't have `isearch-open-invisible' set.  It is unclear
+;    (if (eq char search-yank-word-char)
-;; what's supposed to happen if only a part of [BEG, END) overlaps
+;	(setq isearch-message (if isearch-forward "Word search: "
-;; the extent.
+;				"Word search backward: "))
-(let (to-be-unhidden)
+;      ;; Otherwise let that 1 char be part of the search string.
-(if (map-extents
+;      (setq unread-command-event (character-to-event char))
-	 (lambda (extent ignored)
+;      )
-	   (if (and (<= (extent-start-position extent) beg)
+;    (setq function
-		    (>= (extent-end-position extent) end))
+;	  (if (eq char search-yank-word-char)
-	       ;; All of the region is covered by the extent.
+;	      (if isearch-forward 'word-search-forward 'word-search-backward)
-	       (if (and (eq search-invisible 'open)
+;	    (if isearch-regexp
-			(get extent 'isearch-open-invisible))
+;		(if isearch-forward 're-search-forward 're-search-backward)
-		   (progn
+;	      (if isearch-forward 'search-forward 'search-backward))))
-		     (push extent to-be-unhidden)
+;    ;; Read the search string with corrected prompt.
-		     nil)		; keep mapping
+;    (setq isearch-string (read-string isearch-message isearch-string))
-		 ;; We can't or won't unhide this extent, so we must
+;    ;; Empty means use default.
-		 ;; skip the whole match.  We return from map-extents
+;    (if (= 0 (length isearch-string))
-		 ;; immediately.
+;	(setq isearch-string search-last-string)
-		 t)
+;      ;; Set last search string now so it is set even if we fail.
-	     ;; Else, keep looking.
+;      (setq search-last-string isearch-string))
-	     nil))
+;    ;; Since we used the minibuffer, we should be available for redo.
-	 nil beg end nil 'all-extents-closed 'invisible)
+;    (setq command-history
-	;; The whole match must be skipped.  Signal it by returning t
+;	  (cons (list function isearch-string) command-history))
-	;; to the caller.
+;    ;; Go ahead and search.
-	t
+;    (if search-caps-disable-folding
-;; If any extents need to be unhidden, unhide them.
+;	(setq isearch-case-fold-search
-(mapc #'isearch-unhide-extent to-be-unhidden)
+;	      (no-upper-case-p isearch-string isearch-regexp)))
-;; Will leave this assert for some time, to catch bugs.
+;    (let ((case-fold-search isearch-case-fold-search))
-(assert (null (intersection to-be-unhidden isearch-unhidden-extents)))
+;      (funcall function isearch-string))))
-(setq isearch-unhidden-extents (nconc to-be-unhidden
-					    isearch-unhidden-extents))
-nil)))
-(defun isearch-restore-extent (extent)
-(put extent 'invisible (get extent 'isearch-invisible))
-(put extent 'intangible (get extent 'isearch-intangible))
-(remprop extent 'isearch-invisible)
-(remprop extent 'isearch-intangible))
-;; FSF calls this function `isearch-clean-overlays'.
-(defun isearch-restore-invisible-extents (beg end)
-(cond
-((null beg)
-;; Delete all -- this is called at the end of isearch.
-(mapc #'isearch-restore-extent isearch-unhidden-extents)
-(setq isearch-unhidden-extents nil))
-(t
-;; Extents that do not overlap the match area can be safely
-;; restored to their hidden state.
-(setq isearch-unhidden-extents
-	  (delete-if (lambda (extent)
-		       (unless (extent-in-region-p extent beg end
-						   'all-extents-closed)
-			 (isearch-restore-extent extent)
-			 t))
-		     isearch-unhidden-extents)))))
 (defun isearch-no-upper-case-p (string)
 "Return t if there are no upper case chars in string.
 But upper case chars preceded by \\ do not count since they
 have special meaning in a regexp."
 ;; this incorrectly returns t for "\\\\A"
 (let ((case-fold-search nil))
 (not (string-match "\\(^\\|[^\\]\\)[A-Z]" string))))
 (make-obsolete 'isearch-no-upper-case-p 'no-upper-case-p)
-;; Portability functions to support various Emacs versions.
-(defun isearch-char-to-string (c)
-(if (eventp c)
-(make-string 1 (event-to-character c nil nil t))
-(make-string 1 c)))
-;(defun isearch-text-char-description (c)
-;  (isearch-char-to-string c))
-(define-function 'isearch-text-char-description 'text-char-description)
 ;; Used by etags.el and info.el
 (defmacro with-caps-disable-folding (string &rest body) "\
 Eval BODY with `case-fold-search' let to nil if STRING contains
 uppercase letters and `search-caps-disable-folding' is t."
 ,@body))
 (make-obsolete 'with-caps-disable-folding 'with-search-caps-disable-folding)
 (put 'with-caps-disable-folding 'lisp-indent-function 1)
 (put 'with-caps-disable-folding 'edebug-form-spec '(form body))
-;;;========================================================
-;;; Advanced highlighting
-;; When active, *every* visible match for the current search string is
-;; highlighted: the current one using the normal isearch match color
-;; and all the others using the `isearch-secondary' face.  The extra
-;; highlighting makes it easier to anticipate where the cursor will
-;; land each time you press C-s or C-r to repeat a pending search.
-;; Only the matches visible at any point are highlighted -- when you
-;; move through the buffer, the highlighting is readjusted.
-;; This is based on ideas from Bob Glickstein's `ishl' package.  It
-;; has been merged with XEmacs by Darryl Okahata, and then completely
-;; rewritten by Hrvoje Niksic.
-;; The code makes the following assumptions about the rest of this
-;; file, so be careful when modifying it.
-;; * `isearch-highlight-all-update' should get called when the search
-;;   string changes, or when the search advances.  This is done from
-;;   `isearch-update'.
-;; * `isearch-highlight-all-cleanup' should get called when the search
-;;   is done.  This is performed in `isearch-done'.
-;; * `isearch-string' is expected to contain the current search string
-;;   as entered by the user.
-;; * `isearch-opoint' is expected to contain the location where the
-;;   current search began.
-;; * the type of the current search is expected to be given by
-;;   `isearch-word' and `isearch-regexp'.
-;; * the variable `isearch-invalid-regexp' is expected to be true iff
-;;   `isearch-string' is an invalid regexp.
-(defcustom isearch-highlight-all-matches search-highlight
-"*Non-nil means highlight all visible matches."
-:type 'boolean
-:group 'isearch)
-;; We can't create this face here, as isearch.el is preloaded.
-;; #### Think up a better name for this!
-;(defface isearch-secondary '((t (:foreground "red3")))
-;  "Face to use for highlighting all matches."
-;  :group 'isearch)
-(defvar isearch-highlight-extents nil)
-(defvar isearch-window-start nil)
-(defvar isearch-window-end nil)
-;; We compare isearch-string and isearch-case-fold-search to saved
-;; values for better efficiency.
-(defvar isearch-highlight-last-string nil)
-(defvar isearch-highlight-last-case-fold-search nil)
-(defvar isearch-highlight-last-regexp nil)
-(defun isearch-delete-extents-in-range (start end)
-;; Delete all highlighting extents that overlap [START, END).
-(setq isearch-highlight-extents
-	(delete-if (lambda (extent)
-		     (when (extent-in-region-p extent start end)
-		       (delete-extent extent)
-		       t))
-		   isearch-highlight-extents)))
-(defun isearch-highlight-all-cleanup ()
-;; Stop lazily highlighting and remove extra highlighting from
-;; buffer.
-(mapc #'delete-extent isearch-highlight-extents)
-(setq isearch-highlight-extents nil)
-(setq isearch-window-end nil
-	isearch-highlight-last-string nil))
-(defun isearch-highlight-all-update ()
-;; Update the highlighting if necessary.  This needs to check if the
-;; search string has changed, or if the window has changed position
-;; in the buffer.
-(let ((need-start-over nil))
-;; NB: we don't check for isearch-success because if the point is
-;; after the last match, the search can be unsuccessful, and yet
-;; there are things to highlight.
-(cond ((not isearch-highlight-all-matches))
-	  ((or (equal isearch-string "")
-	       isearch-invalid-regexp)
-	   (isearch-highlight-all-cleanup))
-	  ((not (eq isearch-case-fold-search
-		    isearch-highlight-last-case-fold-search))
-	   ;; This case is usually caused by search string being
-	   ;; changed, which would be caught below, but it can also be
-	   ;; tripped using isearch-toggle-case-fold.
-	   (setq need-start-over t))
-	  ((not (eq isearch-regexp isearch-highlight-last-regexp))
-	   ;; Ditto for isearch-toggle-regexp.
-	   (setq need-start-over t))
-	  ((equal isearch-string isearch-highlight-last-string)
-	   ;; The search string is the same.  We need to do something
-	   ;; if our position has changed.
-	   ;; It would be nice if we didn't have to do this; however,
-	   ;; window-start doesn't support a GUARANTEE flag, so we must
-	   ;; force redisplay to get the correct value for start and end
-	   ;; of window.
-	   (sit-for 0)
-	   ;; Check whether our location has changed.
-	   (let ((start (window-start))
-		 (end (min (window-end) (point-max))))
-	     (cond ((and (= start isearch-window-start)
-			 (= end isearch-window-end))
-		    ;; Our position is unchanged -- do nothing.
-		    )
-		   ((and (> start isearch-window-start)
-			 (> end isearch-window-end)
-			 (<= start isearch-window-end))
-		    ;; We've migrated downward, but we overlap the old
-		    ;; region.  Delete the old non-overlapping extents
-		    ;; and fill in the rest.
-		    (isearch-delete-extents-in-range isearch-window-start start)
-		    (isearch-highlightify-region isearch-window-end end)
-		    (setq isearch-window-start start
-			  isearch-window-end   end))
-		   ((and (<= start isearch-window-start)
-			 (<= end isearch-window-end)
-			 (> end isearch-window-start))
-		    ;; We've migrated upward, but we overlap the old
-		    ;; region.  Delete the old non-overlapping extents
-		    ;; and fill in the rest.
-		    (isearch-delete-extents-in-range
-		     end isearch-window-end)
-		    (isearch-highlightify-region start isearch-window-start)
-		    (setq isearch-window-start start
-			  isearch-window-end   end))
-		   (t
-		    ;; The regions don't overlap, or they overlap in a
-		    ;; weird way.
-		    (setq need-start-over t)))))
-	  (t
-	   ;; The search string has changed.
-	   ;; If more input is pending, don't start over because
-	   ;; starting over forces redisplay, and that slows down
-	   ;; typing.
-	   (unless (input-pending-p)
-	     (setq need-start-over t))))
-(when need-start-over
-;; Force redisplay before removing the old extents, in order to
-;; avoid flicker.
-(sit-for 0)
-(isearch-highlight-all-cleanup)
-(setq isearch-window-start (window-start)
-	    isearch-window-end   (min (window-end) (point-max)))
-(isearch-highlightify-region isearch-window-start isearch-window-end))
-(setq isearch-highlight-last-string isearch-string
-	  isearch-highlight-last-case-fold-search isearch-case-fold-search
-	  isearch-highlight-last-regexp isearch-regexp)))
-(defun isearch-highlight-advance (string forwardp)
-;; Search ahead for the next or previous match.  This is the same as
-;; isearch-search, but without the extra baggage.  Maybe it should
-;; be in a separate function.
-(let ((case-fold-search isearch-case-fold-search))
-(funcall (cond (isearch-word (if forwardp
-				     'word-search-forward
-				   'word-search-backward))
-		   (isearch-regexp (if forwardp
-				       're-search-forward
-				     're-search-backward))
-		   (t (if forwardp
-			  'search-forward
-			'search-backward)))
-	     string nil t)))
-(defun isearch-highlightify-region (start end)
-;; Highlight all occurrences of isearch-string between START and
-;; END.  To do this right, we have to search forward as long as
-;; there are matches that overlap [START, END), and then search
-;; backward the same way.
-(save-excursion
-(goto-char isearch-opoint)
-(let ((lastpoint (point)))
-(while (and (isearch-highlight-advance isearch-string t)
-		  (/= lastpoint (point))
-		  (< (match-beginning 0) end))
-	(let ((extent (make-extent (match-beginning 0)
-				   (match-end 0))))
-	  (set-extent-priority extent (1+ mouse-highlight-priority))
-	  (put extent 'face 'isearch-secondary)
-	  (push extent isearch-highlight-extents))
-	(setq lastpoint (point))))
-(goto-char isearch-opoint)
-(let ((lastpoint (point)))
-(while (and (isearch-highlight-advance isearch-string nil)
-		  (/= lastpoint (point))
-		  (>= (match-end 0) start))
-	(let ((extent (make-extent (match-beginning 0)
-				   (match-end 0))))
-	  (set-extent-priority extent (1+ mouse-highlight-priority))
-	  (put extent 'face 'isearch-secondary)
-	  (push extent isearch-highlight-extents))
-	(setq lastpoint (point))))))
 ;;; isearch-mode.el ends here

Mercurial > hg > xemacs-beta

comparison lisp/isearch-mode.el @ 412:697ef44129c6 r21-2-14