Fix Client move constructor bug

Signed-off-by: Rob Walworth <[email protected]>

sdk/main/src/Client.cc

@@ -117,6 +117,7 @@ Client::~Client()
 
 //-----
 Client::Client(Client&& other) noexcept
+  : mImpl(std::make_unique<ClientImpl>())
 {
   moveClient(std::move(other));
 
@@ -436,16 +437,14 @@ void Client::cancelScheduledNetworkUpdate()
 //-----
 void Client::moveClient(Client&& other)
 {
+  // Cancel this Client's network update if one exists.
   cancelScheduledNetworkUpdate();
 
   // If there's a network update thread running in the moved-from Client, it can't be simply moved. Since it still holds
   // a reference to the moved-from Client, the thread must be stopped and restarted in this Client with the remaining
   // time so that the Client reference can be updated to this Client and no longer be pointing to a moved-from Client.
   if (other.mImpl->mNetworkUpdateThread)
   {
-    // Cancel this Client's network update if one exists.
-    cancelScheduledNetworkUpdate();
-
     // Cancel the update.
     other.cancelScheduledNetworkUpdate();