MbedClient: fix copy constructor

Author: facchinm

libraries/Ethernet/src/EthernetClient.h

@@ -29,9 +29,6 @@ class EthernetClient : public MbedClient {
     NetworkInterface *getNetwork() {
       return Ethernet.getNetwork();
     }
-public:
-    EthernetClient(EthernetClient* orig) : MbedClient(orig) {}
-    EthernetClient() : MbedClient() {}
 };
 
 }

libraries/SocketWrapper/src/MbedClient.h

@@ -41,11 +41,12 @@ class MbedClient : public arduino::Client {
   MbedClient();
 
   // Find something better to save the socket, this is nwither copy nor move constructor
-  MbedClient(MbedClient* orig) {
-    this->sock = orig->sock;
-    orig->borrowed_socket = true;
-    orig->stop();
-    this->setSocket(orig->sock);
+  MbedClient(const MbedClient& orig) {
+    auto _sock = orig.sock;
+    auto _m = (MbedClient*)&orig;
+    _m->borrowed_socket = true;
+    _m->stop();
+    this->setSocket(_sock);
   }
 
   virtual ~MbedClient() {

libraries/WiFi/src/WiFiClient.h

@@ -29,9 +29,6 @@ class WiFiClient : public MbedClient {
     NetworkInterface *getNetwork() {
       return WiFi.getNetwork();
     }
-public:
-    WiFiClient(WiFiClient* orig) : MbedClient(orig) {}
-    WiFiClient() : MbedClient() {}
 };
 
 }