From f9e4d2574c1293edca4bffadedf267f35f5443cb Mon Sep 17 00:00:00 2001
From: Noah Laptop <noah@koontzs.com>
Date: Mon, 30 Dec 2019 08:00:00 -0800
Subject: [PATCH] fix incorrect cast from bool to int, and an invalid return
 value

---
 src/SSLClient.cpp | 9 +++------
 src/SSLClient.h   | 2 +-
 2 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/src/SSLClient.cpp b/src/SSLClient.cpp
index 3529144..0b09baa 100644
--- a/src/SSLClient.cpp
+++ b/src/SSLClient.cpp
@@ -89,7 +89,7 @@ int SSLClient::connect(IPAddress ip, uint16_t port) {
     // connection check
     if (get_arduino_client().connected()) {
         m_error("Cannot have two connections at the same time! Please create another SSLClient instance.", func_name);
-        return -1;
+        return 0;
     }
     // reset indexs for saftey
     m_write_idx = 0;
@@ -112,16 +112,13 @@ int SSLClient::connect(const char *host, uint16_t port) {
     // connection check
     if (get_arduino_client().connected()) {
         m_error("Cannot have two connections at the same time! Please create another SSLClient instance.", func_name);
-        return -1;
+        return 0;
     }
     // reset indexs for saftey
     m_write_idx = 0;
-    // first, if we have a session, check if we're trying to resolve the same host
-    // as before
-    const bool connect_ok = get_arduino_client().connect(host, port);
     // first we need our hidden client member to negotiate the socket for us,
     // since most times socket functionality is implemented in hardeware.
-    if (!connect_ok) {
+    if (!get_arduino_client().connect(host, port)) {
         m_error("Failed to connect using m_client. Are you connected to the internet?", func_name);
         setWriteError(SSL_CLIENT_CONNECT_FAIL);
         return 0;
diff --git a/src/SSLClient.h b/src/SSLClient.h
index 79d6827..28fc932 100644
--- a/src/SSLClient.h
+++ b/src/SSLClient.h
@@ -165,7 +165,7 @@ public:
      * 
      * The implementation for this function can be found in SSLClientImpl::connect_impl(const char*, uint16_t)
      * 
-    * @pre The underlying client object (passed in through the constructor) is in a non-
+     * @pre The underlying client object (passed in through the constructor) is in a non-
      * error state, and must be able to access the IP.
      * @pre SSLClient can only have one connection at a time, so the client
      * object must not already be connected.