diff options
author | Ben Noordhuis <info@bnoordhuis.nl> | 2016-03-31 12:47:06 +0200 |
---|---|---|
committer | Ben Noordhuis <info@bnoordhuis.nl> | 2016-04-05 11:37:02 +0200 |
commit | a7581d0859ce5139b2d7795c5ac1f8df0e85815c (patch) | |
tree | eeb93d30f9690d1ee6e96a46685574aeba25ca9d /src/udp_wrap.cc | |
parent | 00c876dec54f9c515aeed78c75ce61a4e0206d95 (diff) | |
download | android-node-v8-a7581d0859ce5139b2d7795c5ac1f8df0e85815c.tar.gz android-node-v8-a7581d0859ce5139b2d7795c5ac1f8df0e85815c.tar.bz2 android-node-v8-a7581d0859ce5139b2d7795c5ac1f8df0e85815c.zip |
src: replace ARRAY_SIZE with typesafe arraysize
To prevent `ARRAY_SIZE(&arg)` (i.e., taking the array size of a pointer)
from happening again.
PR-URL: https://github.com/nodejs/node/pull/5969
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'src/udp_wrap.cc')
-rw-r--r-- | src/udp_wrap.cc | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/udp_wrap.cc b/src/udp_wrap.cc index 075800d3a7..dc2804812f 100644 --- a/src/udp_wrap.cc +++ b/src/udp_wrap.cc @@ -270,7 +270,7 @@ void UDPWrap::DoSend(const FunctionCallbackInfo<Value>& args, int family) { uv_buf_t bufs_[16]; uv_buf_t* bufs = bufs_; - if (ARRAY_SIZE(bufs_) < count) + if (arraysize(bufs_) < count) bufs = new uv_buf_t[count]; // construct uv_buf_t array @@ -406,14 +406,14 @@ void UDPWrap::OnRecv(uv_udp_t* handle, if (nread < 0) { if (buf->base != nullptr) free(buf->base); - wrap->MakeCallback(env->onmessage_string(), ARRAY_SIZE(argv), argv); + wrap->MakeCallback(env->onmessage_string(), arraysize(argv), argv); return; } char* base = static_cast<char*>(realloc(buf->base, nread)); argv[2] = Buffer::New(env, base, nread).ToLocalChecked(); argv[3] = AddressToJS(env, addr); - wrap->MakeCallback(env->onmessage_string(), ARRAY_SIZE(argv), argv); + wrap->MakeCallback(env->onmessage_string(), arraysize(argv), argv); } |