Assorted jmslib and jakartalib fixups

Author: jiridanek

jakartalib/src/main/java/com/redhat/mqe/lib/ReceiverClient.java

@@ -155,7 +155,7 @@ private void unsubscribe() {
         try {
             session.unsubscribe(durableSubscriberName);
         } catch (JMSException e) {
-            LOG.error("Error while unsubscribing durable subscriptor " + durableSubscriberName);
+            LOG.error("Error while unsubscribing durable subscriber " + durableSubscriberName);
             e.printStackTrace();
         } finally {
             close(session);

jakartalib/src/main/java/com/redhat/mqe/lib/SenderClient.java

@@ -165,7 +165,7 @@ public void startClient() {
                 doTransaction(session, senderOptions.getOption(ClientOptions.TX_ENDLOOP_ACTION).getValue());
             }
         } catch (JMSException | IllegalArgumentException jmse) {
-            LOG.error("Error while sending a message!", jmse.getMessage());
+            LOG.error("Error while sending a message! {}", jmse.getMessage());
             jmse.printStackTrace();
             System.exit(1);
         } finally {
@@ -185,8 +185,7 @@ public void startClient() {
     protected static void setMessageProducer(ClientOptions senderOptions, MessageProducer producer) {
         try {
             // set delivery mode - durable/non-durable
-            String deliveryModeArg = senderOptions.getOption(ClientOptions.MSG_DURABLE).getValue().toLowerCase();
-            int deliveryMode = (deliveryModeArg.equals("true") || deliveryModeArg.equals("yes"))
+            int deliveryMode = Utils.convertOptionToBoolean(senderOptions.getOption(ClientOptions.MSG_DURABLE).getValue())
                 ? DeliveryMode.PERSISTENT : DeliveryMode.NON_PERSISTENT;
             producer.setDeliveryMode(deliveryMode);
             // set time to live of message if provided

jmslib/src/main/java/com/redhat/mqe/lib/SenderClient.java

@@ -165,7 +165,7 @@ public void startClient() {
                 doTransaction(session, senderOptions.getOption(ClientOptions.TX_ENDLOOP_ACTION).getValue());
             }
         } catch (JMSException | IllegalArgumentException jmse) {
-            LOG.error("Error while sending a message!", jmse.getMessage());
+            LOG.error("Error while sending a message! {}", jmse.getMessage());
             jmse.printStackTrace();
             System.exit(1);
         } finally {
@@ -185,8 +185,7 @@ public void startClient() {
     protected static void setMessageProducer(ClientOptions senderOptions, MessageProducer producer) {
         try {
             // set delivery mode - durable/non-durable
-            String deliveryModeArg = senderOptions.getOption(ClientOptions.MSG_DURABLE).getValue().toLowerCase();
-            int deliveryMode = (deliveryModeArg.equals("true") || deliveryModeArg.equals("yes"))
+            int deliveryMode = Utils.convertOptionToBoolean(senderOptions.getOption(ClientOptions.MSG_DURABLE).getValue())
                 ? DeliveryMode.PERSISTENT : DeliveryMode.NON_PERSISTENT;
             producer.setDeliveryMode(deliveryMode);
             // set time to live of message if provided