Re: Boolean checks

2015-02-22 Thread Romain Manni-Bucau
Was mainly to make obvious default and logic. ParseBoolean would make sense if supporting more than true/false IMHO and if dupporting the default we want everywhere. That said while there is no regression I dont care. Le 22 févr. 2015 05:41, "Bhupendra" a écrit : > Make sense. > > Bhupendra > S

Re: Boolean checks

2015-02-21 Thread Bhupendra
Make sense. Bhupendra Sent from phone On 22-Feb-2015, at 9:29 AM, Daniel Cunha wrote: > 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

Boolean checks

2015-02-21 Thread Daniel Cunha
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".e