ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Antoine Levy-Lambert <anto...@gmx.de>
Subject Re: svn commit: r349276 - /ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/LoadResource.java
Date Mon, 28 Nov 2005 21:29:58 GMT
Hello,
I think I am the original culprit for isExists().
Sorry for the ugly looking method name.

Cheers,

Antoine

Matt Benson wrote:

>BC.
>
>--- Dominique Devienne <ddevienne@gmail.com> wrote:
>
>  
>
>>>+++
>>>      
>>>
>ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/LoadResource.java
>  
>
>>>+        if (!src.isExists()) {
>>>+            throw new BuildException(src + "
>>>      
>>>
>>doesn't exist");
>>    
>>
>>>+        }
>>>      
>>>
>>Does isExists() sound strange to me only?
>>java.io.File uses just
>>exists(), so couldn't we just use the same name for
>>Resource? Thanks,
>>--DD
>>
>>
>>    
>>


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


Mime
View raw message