GNU bug report logs - #7718
23.2.91; Info-isearch-search fails isearch

Previous Next

To reply to this bug, email your comments to 7718 AT debbugs.gnu.org.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Leo <sdl.web <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 23.2.91; Info-isearch-search fails isearch
Date: Thu, 23 Dec 2010 10:29:31 +0000

Info-isearch-search returns a function that does not implement the
noerror arg(see Info-search), which means it will err even if
deliberately asked not to.

Leo

Message #8 received at 7718 <at> debbugs.gnu.org (full text, mbox):

From: Juri Linkov <juri <at> jurta.org>
To: Leo <sdl.web <at> gmail.com>
Cc: 7718 <at> debbugs.gnu.org
Subject: Re: bug#7718: 23.2.91; Info-isearch-search fails isearch
Date: Thu, 23 Dec 2010 19:28:57 +0000

> Info-isearch-search returns a function that does not implement the
> noerror arg(see Info-search), which means it will err even if
> deliberately asked not to.

Info-isearch-search does not implement the noerror arg because it just passes it
unchanged to Info-search.  Do you mean Info-search should implement noerror for
some non-interactive uses?  What it should do when noerror is non-nil?

Message #11 received at 7718 <at> debbugs.gnu.org (full text, mbox):

From: Leo <sdl.web <at> gmail.com>
To: Juri Linkov <juri <at> jurta.org>
Cc: 7718 <at> debbugs.gnu.org
Subject: Re: bug#7718: 23.2.91; Info-isearch-search fails isearch
Date: Thu, 23 Dec 2010 22:22:12 +0000

On 2010-12-23 19:28 +0000, Juri Linkov wrote:
>> Info-isearch-search returns a function that does not implement the
>> noerror arg(see Info-search), which means it will err even if
>> deliberately asked not to.
>
> Info-isearch-search does not implement the noerror arg because it just passes it
> unchanged to Info-search.  Do you mean Info-search should implement noerror for
> some non-interactive uses?  What it should do when noerror is non-nil?

That's what I meant. Info-search doesn't implement noerror. I think it
should be like the default isearch search functions:
re-/search-forward/backward: Optional argument, if t, means if fail just
return nil (no error).

Thanks.
Leo

Message #14 received at 7718 <at> debbugs.gnu.org (full text, mbox):

From: Juri Linkov <juri <at> jurta.org>
To: Leo <sdl.web <at> gmail.com>
Cc: 7718 <at> debbugs.gnu.org
Subject: Re: bug#7718: 23.2.91; Info-isearch-search fails isearch
Date: Sat, 25 Dec 2010 02:28:49 +0000

>> Info-isearch-search does not implement the noerror arg because it just passes it
>> unchanged to Info-search.  Do you mean Info-search should implement noerror for
>> some non-interactive uses?  What it should do when noerror is non-nil?
>
> That's what I meant. Info-search doesn't implement noerror. I think it
> should be like the default isearch search functions:
> re-/search-forward/backward: Optional argument, if t, means if fail just
> return nil (no error).

But the whole logic of Isearch using Info-search is built on the
assumption that Info-search ignores the noerror arg (that is usually t).
This allow Isearch to fail in the initial Info node
and to continue searching through other nodes without failing.

Message #17 received at 7718 <at> debbugs.gnu.org (full text, mbox):

From: Leo <sdl.web <at> gmail.com>
To: Juri Linkov <juri <at> jurta.org>
Cc: 7718 <at> debbugs.gnu.org
Subject: Re: bug#7718: 23.2.91; Info-isearch-search fails isearch
Date: Sat, 25 Dec 2010 08:39:57 +0000

On 2010-12-25 02:28 +0000, Juri Linkov wrote:
>>> Info-isearch-search does not implement the noerror arg because it just passes it
>>> unchanged to Info-search.  Do you mean Info-search should implement noerror for
>>> some non-interactive uses?  What it should do when noerror is non-nil?
>>
>> That's what I meant. Info-search doesn't implement noerror. I think it
>> should be like the default isearch search functions:
>> re-/search-forward/backward: Optional argument, if t, means if fail just
>> return nil (no error).
>
> But the whole logic of Isearch using Info-search is built on the
> assumption that Info-search ignores the noerror arg (that is usually t).
> This allow Isearch to fail in the initial Info node
> and to continue searching through other nodes without failing.

For users of isearch-search-string with non-nil noerror it will be a
surprise. For example, I use it before actual isearch to see if the
search is to fail (but don't err) and set things up accordingly. I found
it breaks down in info mode due to its ignoring noerror.

But it is fine if it has to remain that way. I will just wrap
isearch-search-string inside ignore-errors.

Leo

Previous Next

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