Author: rajith
Date: Wed Dec 21 01:44:11 2011
New Revision: 1221557
URL: http://svn.apache.org/viewvc?rev=1221557&view=rev
Log:
QPID-3659 The previous if statement for processing boolean props looks a bit
convoluted. Changed it to a more simple and easily readable statement.
Modified:
qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/AMQBrokerDetails.java
Modified:
qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/AMQBrokerDetails.java
URL:
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/AMQBrokerDetails.java?rev=1221557&r1=1221556&r2=1221557&view=diff
==============================================================================
---
qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/AMQBrokerDetails.java
(original)
+++
qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/AMQBrokerDetails.java
Wed Dec 21 01:44:11 2011
@@ -271,14 +271,18 @@ public class AMQBrokerDetails implements
{
if (_options.containsKey(propName))
{
- if (defaultValue)
- {
- return
!_options.get(propName).equalsIgnoreCase("false");
- }
- else
- {
- return Boolean.parseBoolean(_options.get(propName));
- }
+ if (_options.get(propName).equalsIgnoreCase("false"))
+ {
+ return false;
+ }
+ else if (_options.get(propName).equalsIgnoreCase("true"))
+ {
+ return true;
+ }
+ else
+ {
+ return defaultValue;
+ }
}
else
{
---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project: http://qpid.apache.org
Use/Interact: mailto:commits-subscr...@qpid.apache.org
