Re: Lazy Isearch in dired

[Chong Yidong]

Chong Yidong <address@hidden> writes:

>> The following patch fixes this by using a loop like the loop in the
>> function `isearch-search'.  The docstring of isearch-lazy-highlight-search
>> says: "Attempt to do the search exactly the way the pending isearch would.",
>> so no changes in the docstring are needed, because this patch makes it
>> more similar to the main search function.  It also calls 
>> isearch-success-function
>> to skip the matches outside the current Isearch scope.  By default this
>> function skips invisible matches that are useless for lazy highlighting.
>> This also requires setting search-invisible to nil to not match
>> invisible text.
>
> Hmm, your patch looks reasonable, but I'm not sure.  Could it possibly
> lead to slowdowns in some buffers from the additional search loop?
> Maybe we should hold this till after the release.

Ah, I see you already installed it.  That's OK.

Could you do some quick testing to make sure there isn't a slowdown?
(e.g., try searching in a buffer in a way that generates lots of lazy
highlights).