GNU bug report logs - #19710
24.4.51; Isearch is broken in comints which relly on regexp

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Juri Linkov <juri <at> linkov.net>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#19710: closed (24.4.51; Isearch is broken in comints which
 relly on regexp)
Date: Thu, 05 Feb 2015 01:05:02 +0000

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

Your message dated Thu, 05 Feb 2015 02:57:07 +0200
with message-id <87vbjhdvas.fsf <at> mail.linkov.net>
and subject line Re: bug#19710: 24.4.51; Isearch is broken in comints which relly on regexp
has caused the debbugs.gnu.org bug report #19710,
regarding 24.4.51; Isearch is broken in comints which relly on regexp
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
19710: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=19710
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Vitalie Spinu <spinuvit <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 24.4.51; Isearch is broken in comints which relly on regexp
Date: Wed, 28 Jan 2015 10:10:02 +0100

Recent changes to commit to allow for multiline search assume fields
throughout. The common case when comint-use-prompt-regexp is non-nil is
not captured.

To see the problem start shell and issue a couple of commands. Then
(setq comint-use-prompt-regexp t), and add a couple of RETs. Then try M-r.

You should see "Regexp history I-search backward:" several lines above
where it should be and isearch will not work as expected.

If you restart your shell, you will not even see the "Regexp history
I-search backward:" as there are no fields in the buffer.


  Vitalie


-----

In GNU Emacs 24.4.51.2 (x86_64-unknown-linux-gnu, GTK+ Version 3.12.2)
 of 2015-01-28 on galago
Repository revision: 9664defd262252faf037c5fe1ea095f1cc4b308b
Windowing system distributor `The X.Org Foundation', version 11.0.11600000
System Description:	Ubuntu 14.10

[Message part 3 (message/rfc822, inline)]

From: Juri Linkov <juri <at> linkov.net>
To: Vitalie Spinu <spinuvit <at> gmail.com>
Cc: 19710-done <at> debbugs.gnu.org
Subject: Re: bug#19710: 24.4.51;
 Isearch is broken in comints which relly on regexp
Date: Thu, 05 Feb 2015 02:57:07 +0200

>  >> Recent changes to commit to allow for multiline search assume fields
>  >> throughout. The common case when comint-use-prompt-regexp is non-nil is
>  >> not captured.
>  >>
>  >> To see the problem start shell and issue a couple of commands. Then
>  >> (setq comint-use-prompt-regexp t), and add a couple of RETs. Then try M-r.
>  >>
>  >> You should see "Regexp history I-search backward:" several lines above
>  >> where it should be and isearch will not work as expected.
>  >>
>  >> If you restart your shell, you will not even see the "Regexp history
>  >> I-search backward:" as there are no fields in the buffer.
>
>  > Thanks for the report.  This can be fixed by the patch that
>  > reverts the change of callers, and instead modifies
>  > `comint-line-beginning-position' to support multi-line input
>  > for both cases of comint-use-prompt-regexp = t/nil:
>
> Thanks Juri, this patch solves the problem indeed.

I fixed this on the emacs-24 branch because bug#19572 backported
previous changes to emacs-24.

Previous Next

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