GNU bug report logs - #33446
26; `isearch-lazy-highlight-update' in MASTER today

Previous Next

Full log

Message #24 received at 33446-done <at> debbugs.gnu.org (full text, mbox):

From: Drew Adams <drew.adams <at> oracle.com>
To: Juri Linkov <juri <at> linkov.net>
Cc: 33446-done <at> debbugs.gnu.org
Subject: RE: bug#33446: 26; `isearch-lazy-highlight-update' in MASTER today
Date: Wed, 21 Nov 2018 14:50:56 -0800 (PST)

> > Yes, I guessed that finally.  Beyond that, I think it's
> > not intended to be invoked outside of search (i.e.,
> > when `isearch-mode' is nil).  That was my mistake:
> > invoking it in a command that can be used during Isearch
> > or not.
> >
> > I know that wasn't intended as part of the Isearch
> > design.  Now I test for it and avoid calling `i-l-h-u'.
> >
> > (But it might not hurt for `i-l-h-u' itself to test and
> > do nothing if `isearch-mode' is nil.  I'm not requesting
> > that, however.)
> 
> isearch-lazy-highlight-update can't test for non-nil
> isearch-mode, because it's also used in non-isearch cases
> like in query-replace where isearch-mode is nil.

Oh, right.  Is there some other state it can test
for?  As you said, it can only be expected to be
used (i.e., to work) when within an appropriate
context (e.g. following xyz).

> > Feel free to close this bug, if you like.  It's not
> > clear to me whether any code change is needed.
> 
> I thought that your request was for using
> isearch-lazy-highlight-window as an argument
> of window functions in isearch-lazy-highlight-update.
> If you need this, then please create a new request,
> and I'm closing this one.

That's fine.  Thx.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.