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
@@ -66,7 +64,7 @@ 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());
+									   socket_deleter);
 		if (!sock_fd) {
 			rv = false;
 
@@ -100,7 +98,7 @@ 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());
+									   socket_deleter);
 		if (!sock_fd) {
 			return false;
 		}
@@ -161,7 +159,7 @@ 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());
+									   socket_deleter);
 		if (!sock_fd) {
 			goto exit;
 		}
@@ -254,7 +252,7 @@ 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());
+									   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) {