I didn't even look at the patch(es) posted... I just duplicated the bug and
made the fix that was needed. It was simply that value could be null, and
if so it should just default to incrementing by 1 (operationValue) in that
case.
<propertyfile> is flat out confusing though because of all the bells and
whistles it has. At least now that we have a good set of test cases we can
reasonably ensure that we won't break it too badly - its just that this one
issue slipped through somehow. Having 'value' overloaded depending on
whether an operation is happening or not is part of the confusion.
Erik
----- Original Message -----
From: "Diane Holt" <[EMAIL PROTECTED]>
To: "Ant Developers List" <[EMAIL PROTECTED]>
Sent: Tuesday, May 28, 2002 4:59 PM
Subject: Re: cvs commit:
jakarta-ant/src/testcases/org/apache/tools/ant/taskdefs/optional
PropertyFileTest.java
> Guess you decided that was the fix afterall -- okay. Thanks for putting it
through.
>
> =====
> ([EMAIL PROTECTED])
>
>
>
> __________________________________________________
> Do You Yahoo!?
> Yahoo! - Official partner of 2002 FIFA World Cup
> http://fifaworldcup.yahoo.com
>
> --
> To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
>
>
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
