GNU bug report logs - #1848
23.0.60; isearch-forward fails to find colon characters in Info buffers

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 1848 in the body.
You can then email your comments to 1848 AT debbugs.gnu.org in the normal way.

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

Message #5 received at submit <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Brent Goodrick <bgoodr <at> gmail.com>
To: emacs-pretest-bug <at> gnu.org
Subject: 23.0.60; isearch-forward fails to find colon characters in Info buffers
Date: Sat, 10 Jan 2009 09:52:07 -0800

Please write in English if possible, because the Emacs maintainers
usually do not have translators to read other languages for them.

Your bug report will be posted to the emacs-pretest-bug <at> gnu.org mailing list.

Please describe exactly what actions triggered the bug
and the precise symptoms of the bug:

1. Invoke Emacs CVS top of trunk on Debian Linux "testing" built as of today (2009-01-10). Pass
the -q option.
2. Type C-h i to view the Info buffer
3. Type d to get to the top node. Insure that there is an "Emacs"
node listed.
4. Type C-s which should be bound to isearch-forward
5. Type e
6. Type m
7. Type a
8. Type c
9. Type s
10. Type :
11. It fails to find the colon character.
12. Rerun the above steps again on Emacs 22-2 and it works as
expected.
13. Back in Emacs 23, copy and paste all of the Info buffer into some
other new scrap buffer, go to the top of the buffer and repeat steps 1
through 10 and it finds the colon.

Can someone double-check that this isn't a symptom of a more serious
change of behavior in isearch or elsewhere in Emacs 23?

If Emacs crashed, and you have the Emacs process in the gdb debugger,
please include the output from the following gdb commands:
    `bt full' and `xbacktrace'.
If you would like to further debug the crash, please read the file
/home/brentg/emacs_from_source/install/share/emacs/23.0.60/etc/DEBUG for instructions.


In GNU Emacs 23.0.60.1 (x86_64-unknown-linux-gnu, GTK+ Version 2.12.11)
 of 2009-01-10 on hungover
Windowing system distributor `The X.Org Foundation', version 11.0.10402000
configured using `configure  '--with-x-toolkit' '--with-xft' '--prefix=/home/brentg/emacs_from_source/install''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: en_US.UTF-8
  value of $XMODIFIERS: nil
  locale-coding-system: utf-8-unix
  default-enable-multibyte-characters: t

Major mode: Lisp Interaction

Minor modes in effect:
  desktop-save-mode: t
  iswitchb-mode: t
  erc-ring-mode: t
  erc-services-mode: t
  erc-networks-mode: t
  display-time-mode: t
  shell-dirtrack-mode: t
  delete-selection-mode: t
  mouse-wheel-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  global-auto-composition-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: 1
  transient-mark-mode: t

Recent input:
<down-mouse-1> <mouse-1> M-x r e p o r b 0 C-g C-h 
i C-s e m a c s : C-p q M-x r e p o r t - e m <tab> 
b u <tab> <return>

Recent messages:
Loading /home/brentg/emacs_lisp/bg-emacs.el (source)...done
Composing main Info directory...done
Desktop: 18 buffers restored.
For information about GNU Emacs and the GNU system, type C-h C-a.
Raising the Emacs window with: DISPLAY=:0.0 wmctrl -a 67109026 -i
Resizing The Emacs window with: DISPLAY=:0.0 wmctrl -r 67109026 -i -b add,maximized_vert,maximized_horz
Quit
Pushed a window configuration. [2 times]
Mark saved where search started
Pushed a window configuration.

Message #10 received at 1848 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Juri Linkov <juri <at> jurta.org>
To: Brent Goodrick <bgoodr <at> gmail.com>
Cc: 1848 <at> debbugs.gnu.org
Subject: Re: bug#1848: 23.0.60; isearch-forward fails to find colon characters in Info buffers
Date: Mon, 12 Jan 2009 03:04:32 +0200

> 1. Invoke Emacs CVS top of trunk on Debian Linux "testing" built as of
> today (2009-01-10). Pass the -q option.
> 2. Type C-h i to view the Info buffer
> 3. Type d to get to the top node. Insure that there is an "Emacs"
> node listed.
> 4. Type C-s which should be bound to isearch-forward
> 5. Type e
> 6. Type m
> 7. Type a
> 8. Type c
> 9. Type s
> 10. Type :
> 11. It fails to find the colon character.
> 12. Rerun the above steps again on Emacs 22-2 and it works as
> expected.
> 13. Back in Emacs 23, copy and paste all of the Info buffer into some
> other new scrap buffer, go to the top of the buffer and repeat steps 1
> through 10 and it finds the colon.
>
> Can someone double-check that this isn't a symptom of a more serious
> change of behavior in isearch or elsewhere in Emacs 23?

That's correct, in Emacs 23 C-s doesn't find invisible characters
in Info.  This is an intentional change, not a bug.  Please see
http://thread.gmane.org/gmane.emacs.devel/95142

-- 
Juri Linkov
http://www.jurta.org/emacs/

Message #13 received at 1848 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Juri Linkov <juri <at> jurta.org>
Cc: 1848 <at> debbugs.gnu.org, Brent Goodrick <bgoodr <at> gmail.com>
Subject: Re: bug#1848: 23.0.60; isearch-forward fails to find colon characters in Info buffers
Date: Mon, 12 Jan 2009 03:37:03 -0500

Juri Linkov wrote:

> That's correct, in Emacs 23 C-s doesn't find invisible characters
> in Info.  This is an intentional change, not a bug.  Please see
> http://thread.gmane.org/gmane.emacs.devel/95142

I found it a bit confusing that setting search-invisible to t makes no
difference to the behaviour here. Should Info-search-success-function
respect search-invisible?

Message #18 received at 1848 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 1848 <at> debbugs.gnu.org, Juri Linkov <juri <at> jurta.org>,
        Brent Goodrick <bgoodr <at> gmail.com>
Subject: Re: bug#1848: 23.0.60; isearch-forward fails to find colon characters in Info buffers
Date: Mon, 12 Jan 2009 10:21:48 -0500

>> That's correct, in Emacs 23 C-s doesn't find invisible characters
>> in Info.  This is an intentional change, not a bug.  Please see
>> http://thread.gmane.org/gmane.emacs.devel/95142

> I found it a bit confusing that setting search-invisible to t makes no
> difference to the behaviour here. Should Info-search-success-function
> respect search-invisible?

I'd say yes.  Does anyone object?


        Stefan

Message #23 received at 1848 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: "Brent Goodrick" <bgoodr <at> gmail.com>
To: "Stefan Monnier" <monnier <at> iro.umontreal.ca>
Cc: "Glenn Morris" <rgm <at> gnu.org>, 1848 <at> debbugs.gnu.org,
        "Juri Linkov" <juri <at> jurta.org>
Subject: Re: bug#1848: 23.0.60; isearch-forward fails to find colon characters in Info buffers
Date: Mon, 12 Jan 2009 07:25:28 -0800

[Message part 1 (text/plain, inline)]

Sounds reasonable to me.

Brent

On Mon, Jan 12, 2009 at 7:21 AM, Stefan Monnier <monnier <at> iro.umontreal.ca>wrote:

> >> That's correct, in Emacs 23 C-s doesn't find invisible characters
> >> in Info.  This is an intentional change, not a bug.  Please see
> >> http://thread.gmane.org/gmane.emacs.devel/95142
>
> > I found it a bit confusing that setting search-invisible to t makes no
> > difference to the behaviour here. Should Info-search-success-function
> > respect search-invisible?
>
> I'd say yes.  Does anyone object?
>
>
>        Stefan
>

[Message part 2 (text/html, inline)]

Message #28 received at submit <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>, 1848 <at> debbugs.gnu.org
Cc: rgm <at> gnu.org, bgoodr <at> gmail.com, bug-gnu-emacs <at> gnu.org
Subject: Re: bug#1848: 23.0.60;	isearch-forward fails to find colon characters in Info buffers
Date: Mon, 12 Jan 2009 21:05:59 +0200

> From: Stefan Monnier <monnier <at> iro.umontreal.ca>
> Date: Mon, 12 Jan 2009 10:21:48 -0500
> Cc: 1848 <at> emacsbugs.donarmstrong.com, Brent Goodrick <bgoodr <at> gmail.com>
> 
> >> That's correct, in Emacs 23 C-s doesn't find invisible characters
> >> in Info.  This is an intentional change, not a bug.  Please see
> >> http://thread.gmane.org/gmane.emacs.devel/95142
> 
> > I found it a bit confusing that setting search-invisible to t makes no
> > difference to the behaviour here. Should Info-search-success-function
> > respect search-invisible?
> 
> I'd say yes.  Does anyone object?

I don't object, but wouldn't it be confusing to have it "find" the
colon, without the colon being visible?  Perhaps we should make it
temporarily visible?

Message #38 received at submit <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 1848 <at> debbugs.gnu.org, rgm <at> gnu.org, bgoodr <at> gmail.com,
        bug-gnu-emacs <at> gnu.org
Subject: Re: bug#1848: 23.0.60;	isearch-forward fails to find colon characters in Info buffers
Date: Mon, 12 Jan 2009 14:42:35 -0500

>> >> That's correct, in Emacs 23 C-s doesn't find invisible characters
>> >> in Info.  This is an intentional change, not a bug.  Please see
>> >> http://thread.gmane.org/gmane.emacs.devel/95142
>> 
>> > I found it a bit confusing that setting search-invisible to t makes no
>> > difference to the behaviour here. Should Info-search-success-function
>> > respect search-invisible?
>> 
>> I'd say yes.  Does anyone object?

> I don't object, but wouldn't it be confusing to have it "find" the
> colon, without the colon being visible?  Perhaps we should make it
> temporarily visible?

Oh, yes, definitely.  If we can't temporarily show the invisible text,
then we shouldn't obey (eq search-invisible 'open), but we should still
obey (eq search-invisible t).


        Stefan

Message #48 received at 1848 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Juri Linkov <juri <at> jurta.org>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: 1848 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>,
        bgoodr <at> gmail.com
Subject: Re: bug#1848: 23.0.60; isearch-forward fails to find colon characters in Info buffers
Date: Wed, 14 Jan 2009 03:04:45 +0200

>>> > I found it a bit confusing that setting search-invisible to t makes no
>>> > difference to the behaviour here. Should Info-search-success-function
>>> > respect search-invisible?
>>>
>>> I'd say yes.  Does anyone object?
>
>> I don't object, but wouldn't it be confusing to have it "find" the
>> colon, without the colon being visible?  Perhaps we should make it
>> temporarily visible?
>
> Oh, yes, definitely.  If we can't temporarily show the invisible text,
> then we shouldn't obey (eq search-invisible 'open), but we should still
> obey (eq search-invisible t).

Then the following patch does this.  It checks only for
(eq search-invisible t), not for 'open.

BTW, I'm not sure if we should also skip the breadcrumbs line.
I currently have no problems with the search matching at the
breadcrumbs line.  So maybe we should wait for a request
from someone who has a problem with breadcrumbs.

Index: lisp/info.el
===================================================================
RCS file: /sources/emacs/emacs/lisp/info.el,v
retrieving revision 1.557
diff -c -w -b -r1.557 info.el
*** lisp/info.el	10 Jan 2009 10:42:08 -0000	1.557
--- lisp/info.el	14 Jan 2009 00:59:59 -0000
***************
*** 1868,1878 ****
      (let ((backward (< found beg-found)))
        (not
         (or
  	(if backward
  	    (or (text-property-not-all found beg-found 'invisible nil)
  		(text-property-not-all found beg-found 'display nil))
  	  (or (text-property-not-all beg-found found 'invisible nil)
! 	      (text-property-not-all beg-found found 'display nil)))
  	;; Skip node header line
  	(and (save-excursion (forward-line -1)
  			     (looking-at "\^_"))
--- 1868,1879 ----
      (let ((backward (< found beg-found)))
        (not
         (or
+ 	(and (not (eq search-invisible t))
  	     (if backward
  		 (or (text-property-not-all found beg-found 'invisible nil)
  		     (text-property-not-all found beg-found 'display nil))
  	       (or (text-property-not-all beg-found found 'invisible nil)
! 		   (text-property-not-all beg-found found 'display nil))))
  	;; Skip node header line
  	(and (save-excursion (forward-line -1)
  			     (looking-at "\^_"))

-- 
Juri Linkov
http://www.jurta.org/emacs/

Message #53 received at 1848 <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Juri Linkov <juri <at> jurta.org>
Cc: 1848 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>,
        bgoodr <at> gmail.com
Subject: Re: bug#1848: 23.0.60; isearch-forward fails to find colon characters in Info buffers
Date: Tue, 13 Jan 2009 20:51:05 -0500

> Then the following patch does this.  It checks only for
> (eq search-invisible t), not for 'open.

Looks fine.

> BTW, I'm not sure if we should also skip the breadcrumbs line.
> I currently have no problems with the search matching at the
> breadcrumbs line.  So maybe we should wait for a request
> from someone who has a problem with breadcrumbs.

I see no reason to skip the breadcrumbs.


        Stefan

Message #58 received at 1848-done <at> emacsbugs.donarmstrong.com (full text, mbox):

From: Juri Linkov <juri <at> jurta.org>
To: 1848-done <at> debbugs.gnu.org
Cc: bgoodr <at> gmail.com
Subject: Re: bug#1848: 23.0.60; isearch-forward fails to find colon characters in Info buffers
Date: Thu, 15 Jan 2009 01:18:06 +0200

>> Then the following patch does this.  It checks only for
>> (eq search-invisible t), not for 'open.
>
> Looks fine.

Fixed.

-- 
Juri Linkov
http://www.jurta.org/emacs/

Previous Next

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