SocketWrapper: use static instead lambda for sock deleter

Author: leonardocavagnis

libraries/SocketWrapper/SocketWrapper.h

@@ -17,20 +17,18 @@ class ZephyrSocketWrapper {
 	int ssl_sock_temp_char = -1;
 
 	// custom deleter for shared_ptr to close automatically the socket
-	static auto socket_deleter() {
-		return [](int *fd) {
-			if (fd && *fd != -1) {
-				::close(*fd);
-				delete fd;
-			}
-		};
+	static void socket_deleter(int *fd) {
+		if (fd && *fd != -1) {
+			::close(*fd);
+		}
+		delete fd;
 	}
 
 public:
 	ZephyrSocketWrapper() = default;
 
 	ZephyrSocketWrapper(int fd)
-		: sock_fd(std::shared_ptr<int>(fd < 0 ? nullptr : new int(fd), socket_deleter())) {
+		: sock_fd(std::shared_ptr<int>(fd < 0 ? nullptr : new int(fd), socket_deleter)) {
 	}
 
 	~ZephyrSocketWrapper() = default; // socket close managed by shared_ptr
@@ -65,8 +63,8 @@ class ZephyrSocketWrapper {
 		}
 
 		raw_sock_fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
-		sock_fd = std::shared_ptr<int>(raw_sock_fd < 0 ? nullptr : new int(raw_sock_fd),
-									   socket_deleter());
+		sock_fd =
+			std::shared_ptr<int>(raw_sock_fd < 0 ? nullptr : new int(raw_sock_fd), socket_deleter);
 		if (!sock_fd) {
 			rv = false;
 
@@ -99,8 +97,8 @@ class ZephyrSocketWrapper {
 		inet_pton(AF_INET, _host, &addr.sin_addr);
 
 		raw_sock_fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
-		sock_fd = std::shared_ptr<int>(raw_sock_fd < 0 ? nullptr : new int(raw_sock_fd),
-									   socket_deleter());
+		sock_fd =
+			std::shared_ptr<int>(raw_sock_fd < 0 ? nullptr : new int(raw_sock_fd), socket_deleter);
 		if (!sock_fd) {
 			return false;
 		}
@@ -160,8 +158,8 @@ class ZephyrSocketWrapper {
 		}
 
 		raw_sock_fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TLS_1_2);
-		sock_fd = std::shared_ptr<int>(raw_sock_fd < 0 ? nullptr : new int(raw_sock_fd),
-									   socket_deleter());
+		sock_fd =
+			std::shared_ptr<int>(raw_sock_fd < 0 ? nullptr : new int(raw_sock_fd), socket_deleter);
 		if (!sock_fd) {
 			goto exit;
 		}
@@ -253,8 +251,8 @@ class ZephyrSocketWrapper {
 		addr.sin_addr.s_addr = INADDR_ANY;
 
 		raw_sock_fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
-		sock_fd = std::shared_ptr<int>(raw_sock_fd < 0 ? nullptr : new int(raw_sock_fd),
-									   socket_deleter());
+		sock_fd =
+			std::shared_ptr<int>(raw_sock_fd < 0 ? nullptr : new int(raw_sock_fd), socket_deleter);
 		if (!sock_fd) {
 			return false;
 		}

libraries/SocketWrapper/ZephyrClient.h

@@ -11,7 +11,7 @@ class ZephyrClient : public arduino::Client, ZephyrSocketWrapper {
 
 protected:
 	void setSocket(int sock) {
-		sock_fd = std::shared_ptr<int>(sock < 0 ? nullptr : new int(sock), socket_deleter());
+		sock_fd = std::shared_ptr<int>(sock < 0 ? nullptr : new int(sock), socket_deleter);
 		_connected = true;
 	}
 
@@ -98,7 +98,7 @@ class ZephyrClient : public arduino::Client, ZephyrSocketWrapper {
 	}
 
 	operator bool() {
-		return sock_fd;
+		return (sock_fd != nullptr);
 	}
 
 	String remoteIP() {

libraries/SocketWrapper/ZephyrServer.h

@@ -36,7 +36,7 @@ class ZephyrServer : public arduino::Server, ZephyrSocketWrapper {
 	}
 
 	explicit operator bool() {
-		return sock_fd;
+		return (sock_fd != nullptr);
 	}
 
 	ZephyrClient accept(uint8_t *status = nullptr) {