ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stefan Bodewig <bode...@apache.org>
Subject Re: cvs commit: jakarta-ant/src/main/org/apache/tools/ant/helper ProjectHelper2.java
Date Fri, 13 Dec 2002 11:56:31 GMT
Nothing serious, so no reason the revert anything, just picking nits
as usual ...

>   * Copyright (c) 2000-2002 The Apache Software Foundation.

Why 2000-2002 and not 2002 alone?

>           public void popWrapper() {
>               if( wStack.size() > 0 ) 
>                   wStack.removeElementAt( wStack.size() - 1 );
>           }

We want braces in Ant, *everywhere*.  Even for single line ifs.

Stefan

--
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