Re: isearch-forward and Info-search

[Nick Roberts]

 > > The implementation looks clean.  I just suggest that we should simply
 > > replace isearch-invalid-regexp with isearch-error (there can only be one
 > > error at any given time, either a search failure or an invalid regexp, or
 > > ...).
 > 
 > OK.
 > 
 > I also removed `isearch-within-brackets' because it is used nowhere
 > within Emacs.  And added `condition-case' around the call to
 > `isearch-search-fun' in `isearch-lazy-highlight-search' to return nil
 > in case of error which allows `isearch-lazy-highlight-update' to try to
 > highlight matches from the window beginning (needed for Info isearch).

I like the way this works. Thanks for fixing it.

Nick