You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ra...@apache.org on 2011/12/21 02:44:12 UTC
svn commit: r1221557 -
/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/AMQBrokerDetails.java
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-subscribe@qpid.apache.org