Serevr: Allocate socket instances with std::make_shared (#793)

std::move() transfers them fine, but this was an unintentional oversight
after things were changed.

They should (now) just be allocated as shared in the first place.

Author: twostars

src/Server/shared-server/TcpClientSocketManager.h

@@ -18,7 +18,7 @@ class TcpClientSocketManager : public TcpSocketManager
 			if (_inactiveSocketArray[i] != nullptr)
 				continue;
 
-			auto tcpSocket = std::make_unique<T>(std::forward<Args>(args)..., this);
+			auto tcpSocket = std::make_shared<T>(std::forward<Args>(args)..., this);
 			if (tcpSocket == nullptr)
 				return false;
 

src/Server/shared-server/TcpServerSocketManager.h

@@ -18,7 +18,7 @@ class TcpServerSocketManager : public TcpSocketManager
 			if (_inactiveSocketArray[i] != nullptr)
 				continue;
 
-			auto tcpSocket = std::make_unique<T>(std::forward<Args>(args)..., this);
+			auto tcpSocket = std::make_shared<T>(std::forward<Args>(args)..., this);
 			if (tcpSocket == nullptr)
 				return false;