src: simplify uptime and ppid return values

This commit removes extraneous wrapping of return values in Uptime() and GetParentProcessId(). PR-URL: #24562 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Jon Moss <[email protected]>
nodejs · Nov 28, 2018 · 9caad06 · 9caad06
1 parent 3df8633
commit 9caad06
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/node_process.cc b/src/node_process.cc
@@ -54,7 +54,6 @@ using v8::Integer;
 using v8::Isolate;
 using v8::Local;
 using v8::Name;
-using v8::Number;
 using v8::PropertyCallbackInfo;
 using v8::String;
 using v8::Uint32;
@@ -259,7 +258,7 @@ void Uptime(const FunctionCallbackInfo<Value>& args) {
   uv_update_time(env->event_loop());
   uptime = uv_now(env->event_loop()) - prog_start_time;
 
-  args.GetReturnValue().Set(Number::New(env->isolate(), uptime / 1000));
+  args.GetReturnValue().Set(uptime / 1000);
 }
 
 
@@ -790,7 +789,7 @@ void EnvEnumerator(const PropertyCallbackInfo<Array>& info) {
 
 void GetParentProcessId(Local<Name> property,
                         const PropertyCallbackInfo<Value>& info) {
-  info.GetReturnValue().Set(Integer::New(info.GetIsolate(), uv_os_getppid()));
+  info.GetReturnValue().Set(uv_os_getppid());
 }
 
 void GetActiveRequests(const FunctionCallbackInfo<Value>& args) {