Skip to content

Commit 87a0ac7

Browse files
committed
contextify: replace deprecated SetWeak usage
PR-URL: #5392 Reviewed-By: Reviewed-By: bnoordhuis - Ben Noordhuis <info@bnoordhuis.nl>
1 parent 8a94957 commit 87a0ac7

1 file changed

Lines changed: 3 additions & 5 deletions

File tree

src/node_contextify.cc

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ using v8::Uint8Array;
4545
using v8::UnboundScript;
4646
using v8::V8;
4747
using v8::Value;
48-
using v8::WeakCallbackData;
48+
using v8::WeakCallbackInfo;
4949

5050

5151
class ContextifyContext {
@@ -66,7 +66,7 @@ class ContextifyContext {
6666
// Allocation failure or maximum call stack size reached
6767
if (context_.IsEmpty())
6868
return;
69-
context_.SetWeak(this, WeakCallback<Context>);
69+
context_.SetWeak(this, WeakCallback, v8::WeakCallbackType::kParameter);
7070
context_.MarkIndependent();
7171
}
7272

@@ -305,10 +305,8 @@ class ContextifyContext {
305305
}
306306

307307

308-
template <class T>
309-
static void WeakCallback(const WeakCallbackData<T, ContextifyContext>& data) {
308+
static void WeakCallback(const WeakCallbackInfo<ContextifyContext>& data) {
310309
ContextifyContext* context = data.GetParameter();
311-
context->context_.ClearWeak();
312310
delete context;
313311
}
314312

0 commit comments

Comments
 (0)