ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Magesh Umasankar" <umag...@apache.org>
Subject Re: cvs commit: jakarta-ant/src/main/org/apache/tools/ant/taskdefs Jar.java
Date Tue, 05 Mar 2002 14:44:47 GMT
From: "Conor MacNeill" <conor@cortexebusiness.com.au>

> umagesh@apache.org wrote:
> 
> >   -        if (vPath.equalsIgnoreCase("META-INF/MANIFEST.MF"))  {
> >   +        if ("META-INF/MANIFEST.MF".equalsIgnoreCase(vPath))  {
> 
> Magesh,
> 
> I've never understood why people do this in Java. Makes the code less 
> "readable" IMHO. What is the justification?

"A".equals(b) results only in either true or false.
b.equals("A") may result in true, false or NullPointerException.

> 
> Conor
> 

Cheers,
Magesh

***********************************************
*  When two airplanes almost collide why do   *
*  they call it a near miss?? It sounds like  *
*  a near hit to me!!                         *
***********************************************



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