ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Magesh Umasankar" <umag...@apache.org>
Subject Re: Possible bug fix for LoadFile.java
Date Wed, 29 May 2002 18:56:24 GMT
I see what you mean, now.  Yes, it would be
good, I think, to check for its length and
set it only if it is > 0.

Cheers,
Magesh

*********************************************
*  Early to bed, early to rise sounds great *
*  until the alarm goes off.                *
*********************************************
----- Original Message -----
From: "Diane Holt" <holtdl@yahoo.com>
To: "Ant Developers List" <ant-dev@jakarta.apache.org>
Sent: Wednesday, May 29, 2002 2:49 PM
Subject: Re: Possible bug fix for LoadFile.java


> --- Magesh Umasankar <umagesh@apache.org> wrote:
> > Is there any rule that currently states that
> > a property cannot exist with an empty value?
> > If so, then you are correct.  Otherwise, no harm
> > done.
>
> I'm not sure what you mean. You can set a property to the empty string --
> that's what it's doing now. And the harm is that it doing that guarantees
> that the property will always be set, even if <loadfile> didn't actually
> load anything from the file. IOW: I'm using <loadfile> in combination with
> <filterchain>/<linecontainsregexp>, and if the file doesn't contain a line
> that matches my regexp, then nothing's actually getting loaded from the
> file, and the property shouldn't get set. But the way it is currently, the
> property does get set -- to the empty string.
>
> Diane
>
> =====
> (holtdl@yahoo.com)
>
>
>
> __________________________________________________
> Do You Yahoo!?
> Yahoo! - Official partner of 2002 FIFA World Cup
> http://fifaworldcup.yahoo.com
>
> --
> To unsubscribe, e-mail:   <mailto:ant-dev-unsubscribe@jakarta.apache.org>
> For additional commands, e-mail: <mailto:ant-dev-help@jakarta.apache.org>
>


--
To unsubscribe, e-mail:   <mailto:ant-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:ant-dev-help@jakarta.apache.org>


Mime
View raw message