diff options
author | Anna Henningsen <anna@addaleax.net> | 2019-05-26 22:34:51 +0200 |
---|---|---|
committer | Anna Henningsen <anna@addaleax.net> | 2019-05-29 19:28:38 +0200 |
commit | 5026db142ab5d07c7144dfb272512d3388476a77 (patch) | |
tree | bfe8ca1908b0b94aae498b33ecd3174c7033b2f4 /src/tls_wrap.cc | |
parent | 7329c8fe52eb36a238dfdcdd83b8f81034d983fa (diff) | |
download | android-node-v8-5026db142ab5d07c7144dfb272512d3388476a77.tar.gz android-node-v8-5026db142ab5d07c7144dfb272512d3388476a77.tar.bz2 android-node-v8-5026db142ab5d07c7144dfb272512d3388476a77.zip |
src: use ArrayBufferViewContents more frequently
Using `ArrayBufferViewContents` over
`Buffer::Data()`/`Buffer::Length()` or `SPREAD_BUFFER_ARG` has the
advantages of creating fewer individual variables to keep track off,
not being a “magic” macro that creates variables, reducing code size,
and being faster when receiving on-heap TypedArrays.
PR-URL: https://github.com/nodejs/node/pull/27920
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Diffstat (limited to 'src/tls_wrap.cc')
-rw-r--r-- | src/tls_wrap.cc | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/tls_wrap.cc b/src/tls_wrap.cc index cde5419b9c..e9fe969358 100644 --- a/src/tls_wrap.cc +++ b/src/tls_wrap.cc @@ -187,9 +187,9 @@ void TLSWrap::Receive(const FunctionCallbackInfo<Value>& args) { TLSWrap* wrap; ASSIGN_OR_RETURN_UNWRAP(&wrap, args.Holder()); - CHECK(Buffer::HasInstance(args[0])); - char* data = Buffer::Data(args[0]); - size_t len = Buffer::Length(args[0]); + ArrayBufferViewContents<char> buffer(args[0]); + const char* data = buffer.data(); + size_t len = buffer.length(); Debug(wrap, "Receiving %zu bytes injected from JS", len); // Copy given buffer entirely or partiall if handle becomes closed |