diff options
| author | Rod Vagg <rod@vagg.org> | 2017-09-13 19:43:43 +1000 |
|---|---|---|
| committer | Ruben Bridgewater <ruben@bridgewater.de> | 2017-10-02 01:16:11 -0300 |
| commit | af171b7ba2934c5ffd6bc2d253bed0e46904d5d3 (patch) | |
| tree | ab5456a434d8e0aff6725f265141da797a2eed1e /deps/cares | |
| parent | 13c74706ef872aa9ed5f17572b6466ce5af26b95 (diff) | |
| download | node-new-af171b7ba2934c5ffd6bc2d253bed0e46904d5d3.tar.gz | |
deps: c-ares float, manual ares_ssize_t definition
c-ares switched to using ares_ssize_t for platform-independent ssize_t,
our GYP usage to include config/<platform>/ares_config.h causes problems
when including gyp as a library in core, i.e. in env.h and cares_wrap.h,
where the defines don't get pulled in properly. This, so far, is the
easiest approach to just making it work nicely--explicitly defining
ares_ssize_t for the different Windows variants and ssize_t for
non-Windows where we don't have a configured type from an ares_config.h.
In all of our non-Windows platforms it is ssize_t anyway so this is
safe.
PR-URL: https://github.com/nodejs/node/pull/15378
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Diffstat (limited to 'deps/cares')
| -rw-r--r-- | deps/cares/include/ares_build.h | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/deps/cares/include/ares_build.h b/deps/cares/include/ares_build.h index f94bb2962a..f5535b3849 100644 --- a/deps/cares/include/ares_build.h +++ b/deps/cares/include/ares_build.h @@ -194,14 +194,16 @@ /* Data type definition of ares_ssize_t. */ #ifdef _WIN32 # ifdef _WIN64 -# define CARES_TYPEOF_ARES_SSIZE_T __int64 + typedef __int64 ares_ssize_t; # else -# define CARES_TYPEOF_ARES_SSIZE_T long + typedef long ares_ssize_t; # endif #else -# define CARES_TYPEOF_ARES_SSIZE_T ssize_t; +# ifdef CARES_TYPEOF_ARES_SSIZE_T + typedef CARES_TYPEOF_ARES_SSIZE_T ares_ssize_t; +# else + typedef ssize_t ares_ssize_t; +# endif #endif -typedef CARES_TYPEOF_ARES_SSIZE_T ares_ssize_t; - #endif /* __CARES_BUILD_H */ |