fix clan format

Author: leonardocavagnis

libraries/SocketWrapper/SocketWrapper.h

@@ -17,20 +17,20 @@ 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 auto socket_deleter() {
+		return [](int *fd) {
+			if (fd && *fd != -1) {
+				::close(*fd);
+				delete fd;
+			}
+		};
+	}
 
 public:
 	ZephyrSocketWrapper() = default;
 
-    ZephyrSocketWrapper(int fd)
-        : sock_fd(std::shared_ptr<int>(new int(fd), socket_deleter())) {}
+	ZephyrSocketWrapper(int fd) : sock_fd(std::shared_ptr<int>(new int(fd), socket_deleter())) {
+	}
 
 	~ZephyrSocketWrapper() = default; // socket close managed by shared_ptr
 
@@ -57,25 +57,33 @@ class ZephyrSocketWrapper {
 		}
 
 		if (ret != 0) {
-			if (res) freeaddrinfo(res);
+			if (res) {
+				freeaddrinfo(res);
+			}
 			return false;
 		}
 
 		int raw_sock_fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
 		if (raw_sock_fd < 0) {
-			if (res) freeaddrinfo(res);
+			if (res) {
+				freeaddrinfo(res);
+			}
 			return false;
 		}
 
 		sock_fd = std::shared_ptr<int>(new int(raw_sock_fd), socket_deleter());
 
 		if (::connect(*sock_fd, res->ai_addr, res->ai_addrlen) < 0) {
 			sock_fd.reset();
-			if (res) freeaddrinfo(res);
+			if (res) {
+				freeaddrinfo(res);
+			}
 			return false;
 		}
 
-		if (res) freeaddrinfo(res);
+		if (res) {
+			freeaddrinfo(res);
+		}
 		return true;
 	}
 
@@ -88,10 +96,12 @@ class ZephyrSocketWrapper {
 		addr.sin_port = htons(port);
 		inet_pton(AF_INET, _host, &addr.sin_addr);
 
-        int raw_sock_fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
-        if (raw_sock_fd < 0) return false;
+		int raw_sock_fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
+		if (raw_sock_fd < 0) {
+			return false;
+		}
 
-        sock_fd = std::shared_ptr<int>(new int(raw_sock_fd), socket_deleter());
+		sock_fd = std::shared_ptr<int>(new int(raw_sock_fd), socket_deleter());
 
 		if (::connect(*sock_fd, (struct sockaddr *)&addr, sizeof(addr)) < 0) {
 			sock_fd.reset();
@@ -134,42 +144,54 @@ class ZephyrSocketWrapper {
 		}
 
 		if (ret != 0) {
-			if (res) freeaddrinfo(res);
+			if (res) {
+				freeaddrinfo(res);
+			}
 			return false;
 		}
 
 		if (ca_certificate_pem != nullptr) {
 			ret = tls_credential_add(CA_CERTIFICATE_TAG, TLS_CREDENTIAL_CA_CERTIFICATE,
 									 ca_certificate_pem, strlen(ca_certificate_pem) + 1);
 			if (ret != 0) {
-				if (res) freeaddrinfo(res);
+				if (res) {
+					freeaddrinfo(res);
+				}
 				return false;
 			}
 		}
 
-        int raw_sock_fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TLS_1_2);
-        if (raw_sock_fd < 0) {
-			if (res) freeaddrinfo(res);
+		int raw_sock_fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TLS_1_2);
+		if (raw_sock_fd < 0) {
+			if (res) {
+				freeaddrinfo(res);
+			}
 			return false;
-        }
+		}
 
 		sock_fd = std::shared_ptr<int>(new int(raw_sock_fd), socket_deleter());
 
 		if (setsockopt(*sock_fd, SOL_TLS, TLS_HOSTNAME, host, strlen(host)) ||
 			setsockopt(*sock_fd, SOL_TLS, TLS_SEC_TAG_LIST, sec_tag_opt, sizeof(sec_tag_opt)) ||
 			setsockopt(*sock_fd, SOL_SOCKET, SO_RCVTIMEO, &timeout_opt, sizeof(timeout_opt))) {
-			if (res) freeaddrinfo(res);
+			if (res) {
+				freeaddrinfo(res);
+			}
 			return false;
 		}
 
 		if (::connect(*sock_fd, res->ai_addr, res->ai_addrlen) < 0) {
 			sock_fd.reset();
-			if (res) freeaddrinfo(res);
+			if (res) {
+				freeaddrinfo(res);
+			}
 			return false;
 		}
 		is_ssl = true;
 
-		if (res) freeaddrinfo(res);
+		if (res) {
+			freeaddrinfo(res);
+		}
 		return true;
 	}
 #endif
@@ -199,8 +221,10 @@ class ZephyrSocketWrapper {
 	}
 
 	int recv(uint8_t *buffer, size_t size, int flags = MSG_DONTWAIT) {
-		if (!sock_fd) return -1;
-	
+		if (!sock_fd) {
+			return -1;
+		}
+
 		// TODO: see available()
 		if (ssl_sock_temp_char != -1) {
 			int ret = ::recv(*sock_fd, &buffer[1], size - 1, flags);
@@ -212,7 +236,9 @@ class ZephyrSocketWrapper {
 	}
 
 	int send(const uint8_t *buffer, size_t size) {
-		if (!sock_fd) return -1;
+		if (!sock_fd) {
+			return -1;
+		}
 		return ::send(*sock_fd, buffer, size, 0);
 	}
 
@@ -226,10 +252,12 @@ class ZephyrSocketWrapper {
 		addr.sin_port = htons(port);
 		addr.sin_addr.s_addr = INADDR_ANY;
 
-        int raw_sock_fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
-        if (raw_sock_fd < 0) return false;
+		int raw_sock_fd = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP);
+		if (raw_sock_fd < 0) {
+			return false;
+		}
 
-        sock_fd = std::shared_ptr<int>(new int(raw_sock_fd), socket_deleter());
+		sock_fd = std::shared_ptr<int>(new int(raw_sock_fd), socket_deleter());
 
 		zsock_ioctl(*sock_fd, ZFD_IOCTL_FIONBIO);
 
@@ -242,7 +270,9 @@ class ZephyrSocketWrapper {
 	}
 
 	bool listen(int backlog = 5) {
-		if (!sock_fd) return false;
+		if (!sock_fd) {
+			return false;
+		}
 
 		if (::listen(*sock_fd, backlog) < 0) {
 			sock_fd.reset();
@@ -253,13 +283,17 @@ class ZephyrSocketWrapper {
 	}
 
 	int accept() {
-		if (!sock_fd) return false;
+		if (!sock_fd) {
+			return false;
+		}
 
 		return ::accept(*sock_fd, nullptr, nullptr);
 	}
 
 	String remoteIP() {
-		if (!sock_fd) return {};
+		if (!sock_fd) {
+			return {};
+		}
 
 		struct sockaddr_storage addr;
 		socklen_t addr_len = sizeof(addr);

libraries/SocketWrapper/ZephyrClient.h

@@ -42,7 +42,9 @@ class ZephyrClient : public arduino::Client, ZephyrSocketWrapper {
 #endif
 
 	uint8_t connected() override {
-		if (!sock_fd || *sock_fd == -1) return false;
+		if (!sock_fd || *sock_fd == -1) {
+			return false;
+		}
 
 		uint8_t buf;
 		int ret = ::recv(*sock_fd, &buf, 1, MSG_PEEK | MSG_DONTWAIT);