[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[gnurl] 194/264: cmake: Avoid MSVC C4273 warnings in send/recv checks
From: |
gnunet |
Subject: |
[gnurl] 194/264: cmake: Avoid MSVC C4273 warnings in send/recv checks |
Date: |
Thu, 30 Apr 2020 16:08:17 +0200 |
This is an automated email from the git hooks/post-receive script.
nikita pushed a commit to branch master
in repository gnurl.
commit 89f1e639f6a8d1f4dfb309b95f49028f6c826d47
Author: Brad King <address@hidden>
AuthorDate: Mon Apr 13 09:20:36 2020 -0400
cmake: Avoid MSVC C4273 warnings in send/recv checks
We use `check_c_source_compiles` to check possible send/recv signatures
by reproducing the forward declarations from system headers. On Windows
the `winsock2.h` header adds dll linkage settings to its forward
declaration. If ours does not match the compiler warns:
warning C4273: 'recv': inconsistent dll linkage
Add `WINSOCK_API_LINKAGE` to our test signatures when it is defined so
that our linkage is consistent with that from `winsock2.h`.
Fixes #4764
Closes #5232
---
CMake/OtherTests.cmake | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/CMake/OtherTests.cmake b/CMake/OtherTests.cmake
index f3b92d566..7cec6da6d 100644
--- a/CMake/OtherTests.cmake
+++ b/CMake/OtherTests.cmake
@@ -63,6 +63,9 @@ if(curl_cv_recv)
unset(curl_cv_func_recv_test CACHE)
check_c_source_compiles("
${_source_epilogue}
+ #ifdef WINSOCK_API_LINKAGE
+ WINSOCK_API_LINKAGE
+ #endif
extern ${recv_retv} ${signature_call_conv}
recv(${recv_arg1}, ${recv_arg2}, ${recv_arg3}, ${recv_arg4});
int main(void) {
@@ -127,6 +130,9 @@ if(curl_cv_send)
unset(curl_cv_func_send_test CACHE)
check_c_source_compiles("
${_source_epilogue}
+ #ifdef WINSOCK_API_LINKAGE
+ WINSOCK_API_LINKAGE
+ #endif
extern ${send_retv} ${signature_call_conv}
send(${send_arg1}, ${send_arg2}, ${send_arg3}, ${send_arg4});
int main(void) {
--
To stop receiving notification emails like this one, please contact
address@hidden.
- [gnurl] 247/264: tests/git: ignore mqttd and port files, (continued)
- [gnurl] 247/264: tests/git: ignore mqttd and port files, gnunet, 2020/04/30
- [gnurl] 248/264: docs: fix two typos, gnunet, 2020/04/30
- [gnurl] 240/264: version: skip idn2_check_version() check and add precaution, gnunet, 2020/04/30
- [gnurl] 241/264: lib/mk-ca-bundle: skip empty certs, gnunet, 2020/04/30
- [gnurl] 242/264: transfer: Switch PUT to GET/HEAD on 303 redirect, gnunet, 2020/04/30
- [gnurl] 238/264: curl.h: update comment typo, gnunet, 2020/04/30
- [gnurl] 237/264: GnuTLS: Don't skip really long certificate fields, gnunet, 2020/04/30
- [gnurl] 239/264: RELEASE-NOTES: synced, gnunet, 2020/04/30
- [gnurl] 203/264: sockfilt: tidy variable naming and data structure in select_ws, gnunet, 2020/04/30
- [gnurl] 204/264: tests: run the sws server on "any port", gnunet, 2020/04/30
- [gnurl] 194/264: cmake: Avoid MSVC C4273 warnings in send/recv checks,
gnunet <=
- [gnurl] 195/264: docs/MQTT: replace confusing 80 by 75, gnunet, 2020/04/30
- [gnurl] 193/264: KNOWN_BUGS: Add entry 'Blocking socket operations', gnunet, 2020/04/30
- [gnurl] 215/264: runtests: dummy init the ports variables to avoid warnings, gnunet, 2020/04/30
- [gnurl] 216/264: src: Remove C99 constructs to ensure C89 compliance, gnunet, 2020/04/30
- [gnurl] 252/264: smtp: set auth correctly, gnunet, 2020/04/30
- [gnurl] 250/264: libssh: avoid options override by configuration files, gnunet, 2020/04/30
- [gnurl] 191/264: sockfilt: fix broken pipe on Windows to be ready in select_ws, gnunet, 2020/04/30
- [gnurl] 224/264: mqtt: improve the state machine, gnunet, 2020/04/30
- [gnurl] 219/264: add more alt-svc test coverage, gnunet, 2020/04/30
- [gnurl] 218/264: test1247: use http server to get the port number set, gnunet, 2020/04/30