> [EMAIL PROTECTED] 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?

If vPath is null, you get "false" rather than a NullPointerException.

Note that what *isn't* needed is the nasty:

if (5==i)

which is often encountered in C.

Jon

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to