]> git.ipfire.org Git - thirdparty/curl.git/commitdiff
warnless: replace `read()`/`write()` wrapper functions with macros (Windows)
authorViktor Szakats <commit@vsz.me>
Sat, 14 Jun 2025 10:43:20 +0000 (12:43 +0200)
committerViktor Szakats <commit@vsz.me>
Sun, 15 Jun 2025 00:02:28 +0000 (02:02 +0200)
Map them to `_read()`/`_write()` (with underscore) to avoid recursive
mapping and to use the non-compatibility naming on Windows.

https://learn.microsoft.com/cpp/c-runtime-library/reference/read
https://learn.microsoft.com/cpp/c-runtime-library/reference/write

Follow-up to 6239146e931fd3127f6994975a56d1b4884a708a

Closes #17619

lib/curlx/warnless.c
lib/curlx/warnless.h

index 5ca92450d8935d46d8525a250c5cc731848ccc94..8e3fa1710788f171cba2e405c74ae79c415442c4 100644 (file)
@@ -296,20 +296,6 @@ size_t curlx_sitouz(int sinum)
 #endif
 }
 
-#ifdef _WIN32
-
-ssize_t curlx_read(int fd, void *buf, size_t count)
-{
-  return (ssize_t)read(fd, buf, curlx_uztoui(count));
-}
-
-ssize_t curlx_write(int fd, const void *buf, size_t count)
-{
-  return (ssize_t)write(fd, buf, curlx_uztoui(count));
-}
-
-#endif /* _WIN32 */
-
 /* Ensure that warnless.h redefinitions continue to have an effect
    in "unity" builds. */
 #undef HEADER_CURL_WARNLESS_H_REDEFS
index c78b61169bf8d5d6d33bff09d6fd2c92c48efc4c..c105feb87ed72043fc6b897d5a3c1fe567f6e809 100644 (file)
@@ -57,14 +57,6 @@ unsigned short curlx_uitous(unsigned int uinum);
 
 size_t curlx_sitouz(int sinum);
 
-#ifdef _WIN32
-
-ssize_t curlx_read(int fd, void *buf, size_t count);
-
-ssize_t curlx_write(int fd, const void *buf, size_t count);
-
-#endif /* _WIN32 */
-
 #endif /* HEADER_CURL_WARNLESS_H */
 
 #ifndef HEADER_CURL_WARNLESS_H_REDEFS
@@ -72,9 +64,9 @@ ssize_t curlx_write(int fd, const void *buf, size_t count);
 
 #ifdef _WIN32
 #undef  read
-#define read(fd, buf, count)  curlx_read(fd, buf, count)
+#define read(fd, buf, count)  (ssize_t)_read(fd, buf, curlx_uztoui(count))
 #undef  write
-#define write(fd, buf, count) curlx_write(fd, buf, count)
+#define write(fd, buf, count) (ssize_t)_write(fd, buf, curlx_uztoui(count))
 #endif
 
 #endif /* HEADER_CURL_WARNLESS_H_REDEFS */