From bdec4d9fca99a34145da671b8f8aaf8ffbb4a952 Mon Sep 17 00:00:00 2001 From: cjihrig Date: Sat, 11 Nov 2017 21:10:13 -0500 Subject: [PATCH] src: fix compiler warning in process.ppid This commit fixes a compiler warning with SetAccessor() by passing in the context. PR-URL: https://github.com/nodejs/node/pull/16958 Reviewed-By: Anna Henningsen Reviewed-By: Gireesh Punathil Reviewed-By: Timothy Gu Reviewed-By: James M Snell Reviewed-By: Daniel Bevenius Reviewed-By: Franziska Hinkelmann --- src/node.cc | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/node.cc b/src/node.cc index 8d4259efb303eb..6b0b07f9f38c8f 100644 --- a/src/node.cc +++ b/src/node.cc @@ -3571,8 +3571,9 @@ void SetupProcessObject(Environment* env, READONLY_PROPERTY(process, "pid", Integer::New(env->isolate(), getpid())); READONLY_PROPERTY(process, "features", GetFeatures(env)); - process->SetAccessor(FIXED_ONE_BYTE_STRING(env->isolate(), "ppid"), - GetParentProcessId); + CHECK(process->SetAccessor(env->context(), + FIXED_ONE_BYTE_STRING(env->isolate(), "ppid"), + GetParentProcessId).FromJust()); auto need_immediate_callback_string = FIXED_ONE_BYTE_STRING(env->isolate(), "_needImmediateCallback");