Properly check for connecting or connected status before reconnecting

Note, in our context, is_connecting() includes the resolving part as well as
the actual connection (if we are using c-ares)

fix #3048

2 files changed, 2 insertions, 2 deletions

diff --git a/louloulibs b/louloulibs
-Subproject b53ae922f48f1465a7fa61136f65ec39e38a452
+Subproject 6c812cd86e31569db61cac4e30f77e296d20719
diff --git a/src/irc/irc_client.cpp b/src/irc/irc_client.cpp
index 694baf8..b0ce93a 100644
--- a/src/irc/irc_client.cpp
+++ b/src/irc/irc_client.cpp
@@ -55,8 +55,8 @@ IrcClient::~IrcClient()
 
 void IrcClient::start()
 {
-  if (this->connected || this->connecting)
-    return ;
+  if (this->is_connecting() || this->is_connected())
+    return;
   std::string port;
   bool tls;
   std::tie(port, tls) = this->ports_to_try.top();