Skip to content

Commit 8841947

Browse files
committed
src: rename Buffer::Use() to Buffer::New()
Fixes: #2308 PR-URL: #2352 Reviewed-By: Fedor Indutny <[email protected]> Reviewed-By: Trevor Norris <[email protected]>
1 parent 9a3929d commit 8841947

6 files changed

+11
-11
lines changed

src/node_buffer.cc

+2-2
Original file line numberDiff line numberDiff line change
@@ -365,13 +365,13 @@ MaybeLocal<Object> New(Isolate* isolate, char* data, size_t length) {
365365
Environment* env = Environment::GetCurrent(isolate);
366366
EscapableHandleScope handle_scope(env->isolate());
367367
Local<Object> obj;
368-
if (Buffer::Use(env, data, length).ToLocal(&obj))
368+
if (Buffer::New(env, data, length).ToLocal(&obj))
369369
return handle_scope.Escape(obj);
370370
return Local<Object>();
371371
}
372372

373373

374-
MaybeLocal<Object> Use(Environment* env, char* data, size_t length) {
374+
MaybeLocal<Object> New(Environment* env, char* data, size_t length) {
375375
EscapableHandleScope scope(env->isolate());
376376

377377
if (length > 0) {

src/node_crypto.cc

+5-5
Original file line numberDiff line numberDiff line change
@@ -3006,7 +3006,7 @@ void CipherBase::GetAuthTag(const FunctionCallbackInfo<Value>& args) {
30063006
unsigned int out_len = 0;
30073007

30083008
if (cipher->GetAuthTag(&out, &out_len)) {
3009-
Local<Object> buf = Buffer::Use(env, out, out_len).ToLocalChecked();
3009+
Local<Object> buf = Buffer::New(env, out, out_len).ToLocalChecked();
30103010
args.GetReturnValue().Set(buf);
30113011
} else {
30123012
env->ThrowError("Attempting to get auth tag in unsupported state");
@@ -4481,7 +4481,7 @@ void ECDH::ComputeSecret(const FunctionCallbackInfo<Value>& args) {
44814481
return env->ThrowError("Failed to compute ECDH key");
44824482
}
44834483

4484-
Local<Object> buf = Buffer::Use(env, out, out_len).ToLocalChecked();
4484+
Local<Object> buf = Buffer::New(env, out, out_len).ToLocalChecked();
44854485
args.GetReturnValue().Set(buf);
44864486
}
44874487

@@ -4519,7 +4519,7 @@ void ECDH::GetPublicKey(const FunctionCallbackInfo<Value>& args) {
45194519
}
45204520

45214521
Local<Object> buf =
4522-
Buffer::Use(env, reinterpret_cast<char*>(out), size).ToLocalChecked();
4522+
Buffer::New(env, reinterpret_cast<char*>(out), size).ToLocalChecked();
45234523
args.GetReturnValue().Set(buf);
45244524
}
45254525

@@ -4546,7 +4546,7 @@ void ECDH::GetPrivateKey(const FunctionCallbackInfo<Value>& args) {
45464546
}
45474547

45484548
Local<Object> buf =
4549-
Buffer::Use(env, reinterpret_cast<char*>(out), size).ToLocalChecked();
4549+
Buffer::New(env, reinterpret_cast<char*>(out), size).ToLocalChecked();
45504550
args.GetReturnValue().Set(buf);
45514551
}
45524552

@@ -4949,7 +4949,7 @@ void RandomBytesCheck(RandomBytesRequest* req, Local<Value> argv[2]) {
49494949
size_t size;
49504950
req->return_memory(&data, &size);
49514951
argv[0] = Null(req->env()->isolate());
4952-
argv[1] = Buffer::Use(req->env(), data, size).ToLocalChecked();
4952+
argv[1] = Buffer::New(req->env(), data, size).ToLocalChecked();
49534953
}
49544954
}
49554955

src/node_internals.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -284,7 +284,7 @@ v8::MaybeLocal<v8::Object> New(Environment* env,
284284
// Takes ownership of |data|. Must allocate |data| with malloc() or realloc()
285285
// because ArrayBufferAllocator::Free() deallocates it again with free().
286286
// Mixing operator new and free() is undefined behavior so don't do that.
287-
v8::MaybeLocal<v8::Object> Use(Environment* env, char* data, size_t length);
287+
v8::MaybeLocal<v8::Object> New(Environment* env, char* data, size_t length);
288288
} // namespace Buffer
289289

290290
} // namespace node

src/stream_wrap.cc

+1-1
Original file line numberDiff line numberDiff line change
@@ -223,7 +223,7 @@ void StreamWrap::OnReadImpl(ssize_t nread,
223223
CHECK_EQ(pending, UV_UNKNOWN_HANDLE);
224224
}
225225

226-
Local<Object> obj = Buffer::Use(env, base, nread).ToLocalChecked();
226+
Local<Object> obj = Buffer::New(env, base, nread).ToLocalChecked();
227227
wrap->EmitData(nread, obj, pending_obj);
228228
}
229229

src/tls_wrap.cc

+1-1
Original file line numberDiff line numberDiff line change
@@ -660,7 +660,7 @@ void TLSWrap::OnReadSelf(ssize_t nread,
660660
TLSWrap* wrap = static_cast<TLSWrap*>(ctx);
661661
Local<Object> buf_obj;
662662
if (buf != nullptr)
663-
buf_obj = Buffer::Use(wrap->env(), buf->base, buf->len).ToLocalChecked();
663+
buf_obj = Buffer::New(wrap->env(), buf->base, buf->len).ToLocalChecked();
664664
wrap->EmitData(nread, buf_obj, Local<Object>());
665665
}
666666

src/udp_wrap.cc

+1-1
Original file line numberDiff line numberDiff line change
@@ -408,7 +408,7 @@ void UDPWrap::OnRecv(uv_udp_t* handle,
408408
}
409409

410410
char* base = static_cast<char*>(realloc(buf->base, nread));
411-
argv[2] = Buffer::Use(env, base, nread).ToLocalChecked();
411+
argv[2] = Buffer::New(env, base, nread).ToLocalChecked();
412412
argv[3] = AddressToJS(env, addr);
413413
wrap->MakeCallback(env->onmessage_string(), ARRAY_SIZE(argv), argv);
414414
}

0 commit comments

Comments
 (0)