[ jEdit-devel ] [ jedit-Bugs-2207111 ] 4.3pre16: regex $ not matching the last line of a selection

Bugs item #2207111, was opened at 2008-10-29 11:17
Message generated for change (Comment added) made by solrac776
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100588&aid=2207111&group_id=588

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: search and replace
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Carlos (solrac776)
Assigned to: Nobody/Anonymous (nobody)
Summary: 4.3pre16: regex $ not matching the last line of a selection

Initial Comment:
This is reproducible on 4.3pre16-r19000.

Given the below text:

   function make_student_rec(
      p_birth_date  table.birth_date%type,
      p_registration_code  table.ests_code%type,
      p_registration_date  table.ests_date%type
   ) return student_rec as
         v_temp student_rec;
      begin
         p_birth_date  table.birth_date%type,
         p_registration_code  table.ests_code%type,
         p_registration_date  table.ests_date%type
      end;

Reproduce the issue:
   1. Select the 3 lines between 'begin' and 'end'.
   2. Press Ctrl-F to bring up the find and replace dialogue.
      ('regular expressions', 'hypersearch' and 'selection' should be selected.)
   
   3. Search for: p_(\w+)  [^\n]+$
   4. Replace with: v_temp.$1 := p_$1;
   5. Press 'replace all'.
   
   This only matches and replaces the first two lines.
   It should match and replace all three lines.

Maybe of interest:
   Undo the above changes (so that you have your original text again).
   Select the 3 lines between 'begin' and 'end' and also select 'end'.
   (So now we have 4 lines selected)
   
   Repeat steps 2-4 from above.
   
   This works, matching and replacing the three lines above 'end'




----------------------------------------------------------------------

Comment By: Carlos (solrac776)
Date: 2008-10-31 15:01

Message:
Checked on r13969.  Seems to be fixed.  Closing the bug.

----------------------------------------------------------------------

Comment By: Carlos (solrac776)
Date: 2008-10-30 09:47

Message:
My fault.  You are correct k_satoda.  I'm using r13908, must have been
thinking of something else when I wrote r19000!  I will try the latest.

----------------------------------------------------------------------

Comment By: Kazutoshi Satoda (k_satoda)
Date: 2008-10-29 16:05

Message:
I couldn't reproduce this with my build at r13952.

The reported revision r19000 seems not correct because the current trunk
is at r13963. Please verify the revision and try another build newer
than r13908 which is the last fix for regex search.

----------------------------------------------------------------------

Comment By: Carlos (solrac776)
Date: 2008-10-29 11:29

Message:
I think that it is the $ that makes the difference in the search
expression.

If you repeat the above steps, but without the $ inthe search expression,
it matches and replaces all 3 lines.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100588&aid=2207111&group_id=588