Rename RpcClientParams#isUseMandatory to shouldUseMandatory

[#161476914]

Fixes #417

(cherry picked from commit 7f89b26)

Author: acogoluegnes

src/main/java/com/rabbitmq/client/RpcClient.java

@@ -107,7 +107,7 @@ public RpcClient(RpcClientParams params) throws
             throw new IllegalArgumentException("Timeout argument must be NO_TIMEOUT(-1) or non-negative.");
         }
         _timeout = params.getTimeout();
-        _useMandatory = params.isUseMandatory();
+        _useMandatory = params.shouldUseMandatory();
         _replyHandler = params.getReplyHandler();
         _correlationId = 0;
 
@@ -145,17 +145,11 @@ public RpcClient(RpcClientParams params) throws
     @Deprecated
     public RpcClient(Channel channel, String exchange, String routingKey, String replyTo, int timeout) throws
             IOException {
-        _channel = channel;
-        _exchange = exchange;
-        _routingKey = routingKey;
-        _replyTo = replyTo;
-        if (timeout < NO_TIMEOUT) throw new IllegalArgumentException("Timeout argument must be NO_TIMEOUT(-1) or non-negative.");
-        _timeout = timeout;
-        _useMandatory = false;
-        _replyHandler = DEFAULT_REPLY_HANDLER;
-        _correlationId = 0;
-
-        _consumer = setupConsumer();
+        this(new RpcClientParams()
+            .channel(channel).exchange(exchange).routingKey(routingKey)
+            .replyTo(replyTo).timeout(timeout)
+            .useMandatory(false)
+        );
     }
 
     /**

src/main/java/com/rabbitmq/client/RpcClientParams.java

@@ -166,7 +166,7 @@ public RpcClientParams useMandatory() {
         return useMandatory(true);
     }
 
-    public boolean isUseMandatory() {
+    public boolean shouldUseMandatory() {
         return useMandatory;
     }