I saw that we have some cases like this:
"false".equalsIgnoreCase(ejbJar.properties.getProperty("openejb.cdi.activated"))
Doesn't is better use Boolean.parseBoolean()?
With parseBoolean it checks if the string argument is not null and is
equalsIgnoreCase to the "true"
For case like this:
"true".equalsIgnoreCase(SystemInstance.get().getProperty("openejb.cdi.debug",
"false")
not need to set a default value, for null value the result is false.
If we want an inverse logic, we need add only "!"
I think better. IMHO. :)
--
Daniel Cunha (soro)
