Ticket #69 (closed defect: fixed)

Opened 10 years ago

Last modified 5 years ago

savannah: Bad regexp search in editor (" $")

Reported by: bilboq Owned by:
Priority: major Milestone: 4.7
Component: mcedit Version:
Keywords: Cc:
Blocked By: Blocking:
Branch state: no branch Votes for changeset:

Description (last modified by ossi) (diff)

Original: http://savannah.gnu.org/bugs/?19757

Submitted by:Martin Petricek <bilboq>Submitted on:Mon 30 Apr 2007 03:20:29 PM UTC
Category:EditorSeverity:3 - Normal
Status:ConfirmedPrivacy:Public
Assigned to:Pavel Tsekov <ptsekov>Open/Closed:Open
Release:current (CVS or snapshot)Operating System:GNU/Linux

Original submission:

When I used Command -> Find file and entered " $" (space and a 
dollar, meaning one space character at end of line, regular 
expression and case sensitivity is both checked), I find many lines 
in many files. However, when I enter one of the files, move to the 
start of the file and search for the same expression " $" using F7 
("regular expression" checkbox is checked, rest is unchecked), 
nothing is found ("Search string not found"). When I move to the 
line where the space was found by "Find file" the space at end of 
line is there.

To reproduce, file with lines like:

"a word "
"another "
"another "

may be used (without the quotation marks of course, the spaces at 
end of lines are important) 

Comment 1 by Pavel Tsekov <ptsekov> at Wed 02 May 2007 11:53:05 AM UTC:

If you are building MC from source you can apply the attached patch.
 I am working on a patch to fix both this issue and the 1024 
characters line limit described in bug #19329. The final patch may 
or may not include the attached patch, so this is an interim 
solution.

(file #12664)

Comment 2 by Pavel Tsekov <ptsekov> at Wed 02 May 2007 02:35:49 PM UTC:

In fact the code dealing with RE in the editor is pretty bizzare.. I
rewrite would be necessary.

Attachments

12664-string_regexp_search-match-eol.patch (600 bytes) - added by slavazanko 10 years ago.
added by ptsekov

Change History

Changed 10 years ago by slavazanko

added by ptsekov

comment:1 Changed 10 years ago by styx

  • Milestone set to future releases

comment:2 Changed 10 years ago by slavazanko

  • Status changed from new to closed
  • Resolution set to fixed

Bug fixed in new search engine

comment:3 Changed 10 years ago by slavazanko

  • Milestone changed from future releases to 4.7

comment:4 Changed 5 years ago by ossi

  • Description modified (diff)
  • Branch state set to no branch
  • Reporter changed from slavazanko to bilboq
Note: See TracTickets for help on using tickets.