ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Peter Donald <dona...@apache.org>
Subject Re: cvs commit: jakarta-ant/src/main/org/apache/tools/ant ProjectHelper.java Target.java
Date Thu, 27 Sep 2001 11:26:03 GMT
On Thu, 27 Sep 2001 21:27, bodewig@apache.org wrote:
>   1.22.2.1  +4 -1     
> jakarta-ant/src/main/org/apache/tools/ant/Target.java
>
>   Index: Target.java
>   ===================================================================
>   RCS file:
> /home/cvs/jakarta-ant/src/main/org/apache/tools/ant/Target.java,v
> retrieving revision 1.22
>   retrieving revision 1.22.2.1
>   diff -u -r1.22 -r1.22.2.1
>   --- Target.java	2001/08/02 11:42:55	1.22
>   +++ Target.java	2001/09/27 11:27:24	1.22.2.1
>   @@ -85,7 +85,10 @@
>                StringTokenizer tok =
>                    new StringTokenizer(depS, ",", false);
>                while (tok.hasMoreTokens()) {
>   -                addDependency(tok.nextToken().trim());
>   +                String token = tok.nextToken().trim();
>   +                if (!token.equals("")) {
>   +                    addDependency(token);
>   +                }
>                }

Are you sure this is a good idea? This allows dependencies such as "a," to be 
classified as valid which definetly is not the behaviour I would expect.

-- 
Cheers,

Pete

------------------------------------------------
 "No. Try not. Do. Or do not. There is no try." 
                                     -- Yoda 
------------------------------------------------

Mime
View raw message