Though we haven't been following the practise of
always creating a bug report before fixing it,
I think it would be a good approach to include it
in Bugzilla. The reason is some user may come across
this issue before the release and may logically
search Bugzilla to see if the error has already
been reported/addressed before creating a report.
Expecting the user to search through mailing lists
or the README in CVS Head would not be fair.
Cheers,
Magesh
----- Original Message -----
From: "peter reilly" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Wednesday, May 28, 2003 6:57 AM
Subject: Advice required on committing
Hi, I have just been made a ant committer.
I have found a bug in ant's regular expession
handling for replacing matched groups.
Ant uses \n to identify the matched group,
jdk14 and jakarta-oro uses $n to identify the
group. The code in ant correctly replaces the \
with $. However, it does not escape any $ on
the input.
The following
<replaceregex pattern="@([EMAIL PROTECTED])@" replace="${\1}"/>
fails on jdk14 as Matcher#appendReplacement gets passed
"${$1}" rather than "\${$1}".
My question is:
should I make a bugzilla entry or just
commit the changes, with unit test and
appropriate comment ?
Peter
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
