From cc71e037752ec4c36c2b02172a5720d2719f79b8 Mon Sep 17 00:00:00 2001 From: Rajith Muditha Attapattu Date: Wed, 21 Dec 2011 01:44:11 +0000 Subject: QPID-3659 The previous if statement for processing boolean props looks a bit convoluted. Changed it to a more simple and easily readable statement. git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@1221557 13f79535-47bb-0310-9956-ffa450edef68 --- .../org/apache/qpid/client/AMQBrokerDetails.java | 20 ++++++++++++-------- 1 file changed, 12 insertions(+), 8 deletions(-) (limited to 'java/client') diff --git a/java/client/src/main/java/org/apache/qpid/client/AMQBrokerDetails.java b/java/client/src/main/java/org/apache/qpid/client/AMQBrokerDetails.java index d6589bacdf..b343820d80 100644 --- a/java/client/src/main/java/org/apache/qpid/client/AMQBrokerDetails.java +++ b/java/client/src/main/java/org/apache/qpid/client/AMQBrokerDetails.java @@ -271,14 +271,18 @@ public class AMQBrokerDetails implements BrokerDetails { 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 { -- cgit v1.2.1