Bugs item #1819248, was opened at 2007-10-24 13:54
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1819248&group_id=139143
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: core
Group: ver 1.2.x
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Alexander Bergolth (bergolth)
Assigned to: Nobody/Anonymous (nobody)
Summary: replace_all, replace_body_all, subst anchor bug
Initial Comment:
Hi!
The replace_all, replace_body_all and subst functions all fail to match the
beginning of the next line, when the last match ended with a newline.
e.g. when trying to remove the three lines
-------------------- 8< --------------------
a=alt:1 3 : xmv2v5mo ix7lpuPK 137.208.90.150 13726.
a=alt:2 2 : lPFc4bEV udepPIjW 192.168.217.1 13726.
a=alt:3 1 : 4o1caMp2 9P6XBBmd 192.168.172.1 13726.
-------------------- 8< --------------------
... replace_body_all("^a=alt:.*\r\n","") will remove only the first and the
third line, it won't match the second line.
The reason is that after the first match, eflags|=REG_NOTBOL is set, so that in
all subsequent matches, "^" won't match the beginning of the string.
Unfortunately, since the last match ended with a newline, "^" _should_ match at
the beginning of the string in this case.
Maybe the REG_NOTBOL flag should be set depending on the last character before
the next match starts?
This bug is also present in subst_run() of re.c in the openser core.
I've attached a patch that fixes the problem for me.
(Not very well tested.)
Cheers,
--leo
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=743020&aid=1819248&group_id=139143
_______________________________________________
Devel mailing list
Devel@openser.org
http://openser.org/cgi-bin/mailman/listinfo/devel
