ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Diane Holt <>
Subject Re: cvs commit: jakarta-ant/src/testcases/org/apache/tools/ant/taskdefs/optional
Date Tue, 28 May 2002 22:29:16 GMT
--- Erik Hatcher <> wrote:
> I didn't even look at the patch(es) posted... I just duplicated the bug
> and made the fix that was needed.

If you read the bug report you closed with this change, you'll see that
the point of hesitation about putting through this particular change as
the fix is whether value (the variable) should ever really be null at that
point or whether there's a bug in the logic somewhere else along the line
and this approach to preventing the NPE just masks that.



Do You Yahoo!?
Yahoo! - Official partner of 2002 FIFA World Cup

To unsubscribe, e-mail:   <>
For additional commands, e-mail: <>

View raw message