From 4b463992e59ad8338618b71a314d15c20525787a Mon Sep 17 00:00:00 2001 From: Xiang Xiao Date: Tue, 24 Dec 2019 21:28:32 +0800 Subject: lib: fix warnings found when porting to NuttX - Undefine DEBUGASSERT in curl_setup_once.h in case it was already defined as a system macro. - Don't compile write32_le in curl_endian unless CURL_SIZEOF_CURL_OFF_T > 4, since it's only used by Curl_write64_le. - Include in socketpair.c. Closes https://github.com/curl/curl/pull/4756 --- lib/curl_endian.c | 2 +- lib/curl_setup_once.h | 1 + lib/socketpair.c | 3 +++ 3 files changed, 5 insertions(+), 1 deletion(-) diff --git a/lib/curl_endian.c b/lib/curl_endian.c index b7563b3de..a774d136e 100644 --- a/lib/curl_endian.c +++ b/lib/curl_endian.c @@ -81,6 +81,7 @@ unsigned short Curl_read16_be(const unsigned char *buf) ((unsigned short)buf[1])); } +#if (CURL_SIZEOF_CURL_OFF_T > 4) /* * write32_le() * @@ -100,7 +101,6 @@ static void write32_le(const int value, unsigned char *buffer) buffer[3] = (char)((value & 0xFF000000) >> 24); } -#if (CURL_SIZEOF_CURL_OFF_T > 4) /* * Curl_write64_le() * diff --git a/lib/curl_setup_once.h b/lib/curl_setup_once.h index 8db74e9b4..8890f3890 100644 --- a/lib/curl_setup_once.h +++ b/lib/curl_setup_once.h @@ -374,6 +374,7 @@ typedef int sig_atomic_t; * Macro used to include assertion code only in debug builds. */ +#undef DEBUGASSERT #if defined(DEBUGBUILD) && defined(HAVE_ASSERT_H) #define DEBUGASSERT(x) assert(x) #else diff --git a/lib/socketpair.c b/lib/socketpair.c index 1f0e2e4a4..1ec0d75a4 100644 --- a/lib/socketpair.c +++ b/lib/socketpair.c @@ -40,6 +40,9 @@ #ifdef HAVE_NETINET_IN_H #include /* IPPROTO_TCP */ #endif +#ifdef HAVE_ARPA_INET_H +#include +#endif #ifndef INADDR_LOOPBACK #define INADDR_LOOPBACK 0x7f000001 #endif /* !INADDR_LOOPBACK */ -- cgit v1.2.3