Suppress bogus [-Wlogical-op] for (errno == EWOULDBLOCK || errno == EAGAIN)

Author: Girgias

ext/sockets/sockets.c

@@ -1045,11 +1045,15 @@ PHP_FUNCTION(socket_read)
 	if (retval == -1) {
 		/* if the socket is in non-blocking mode and there's no data to read,
 		don't output any error, as this is a normal situation, and not an error */
+
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wlogical-op"
 		if (errno == EAGAIN
 #ifdef EWOULDBLOCK
 		|| errno == EWOULDBLOCK
 #endif
 		) {
+#pragma GCC diagnostic pop
 			php_sock->error = errno;
 			SOCKETS_G(last_error) = errno;
 		} else {

main/streams/plain_wrapper.c

@@ -351,7 +351,10 @@ static ssize_t php_stdiop_write(php_stream *stream, const char *buf, size_t coun
 		ssize_t bytes_written = write(data->fd, buf, count);
 #endif
 		if (bytes_written < 0) {
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wlogical-op"
 			if (errno == EWOULDBLOCK || errno == EAGAIN) {
+#pragma GCC diagnostic pop
 				return 0;
 			}
 			if (errno == EINTR) {
@@ -420,7 +423,10 @@ static ssize_t php_stdiop_read(php_stream *stream, char *buf, size_t count)
 		}
 
 		if (ret < 0) {
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wlogical-op"
 			if (errno == EWOULDBLOCK || errno == EAGAIN) {
+#pragma GCC diagnostic pop
 				/* Not an error. */
 				ret = 0;
 			} else if (errno == EINTR) {

main/streams/xp_socket.c

@@ -75,7 +75,10 @@ static ssize_t php_sockop_write(php_stream *stream, const char *buf, size_t coun
 	if (didwrite <= 0) {
 		char *estr;
 		int err = php_socket_errno();
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wlogical-op"
 		if (err == EWOULDBLOCK || err == EAGAIN) {
+#pragma GCC diagnostic pop
 			if (sock->is_blocked) {
 				int retval;
 
@@ -166,7 +169,10 @@ static ssize_t php_sockop_read(php_stream *stream, char *buf, size_t count)
 	err = php_socket_errno();
 
 	if (nr_bytes < 0) {
-		if (err == EAGAIN || err == EWOULDBLOCK) {
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wlogical-op"
+		if (err == EWOULDBLOCK || err == EAGAIN) {
+#pragma GCC diagnostic pop
 			nr_bytes = 0;
 		} else {
 			stream->eof = 1;

sapi/fpm/fpm/fpm_stdio.c

@@ -166,7 +166,10 @@ static void fpm_stdio_child_said(struct fpm_event_s *ev, short which, void *arg)
 stdio_read:
 		in_buf = read(fd, buf, max_buf_size - 1);
 		if (in_buf <= 0) { /* no data */
+#pragma GCC diagnostic push
+#pragma GCC diagnostic ignored "-Wlogical-op"
 			if (in_buf == 0 || (errno != EAGAIN && errno != EWOULDBLOCK)) {
+#pragma GCC diagnostic pop
 				/* pipe is closed or error */
 				read_fail = (in_buf < 0) ? in_buf : 1;
 			}