diff --git a/src/env.cc b/src/env.cc index 73d1e4fab1207c..312ad292ab8c21 100644 --- a/src/env.cc +++ b/src/env.cc @@ -177,7 +177,7 @@ void Environment::PrintSyncTrace() const { StackTrace::CurrentStackTrace(isolate(), 10, StackTrace::kDetailed); fprintf(stderr, "(node:%u) WARNING: Detected use of sync API\n", - GetProcessId()); + uv_os_getpid()); for (int i = 0; i < stack->GetFrameCount() - 1; i++) { Local stack_frame = stack->GetFrame(i); diff --git a/src/inspector_agent.cc b/src/inspector_agent.cc index 875c12efa02375..48cc29324fbdbb 100644 --- a/src/inspector_agent.cc +++ b/src/inspector_agent.cc @@ -109,7 +109,7 @@ static int StartDebugSignalHandler() { CHECK_EQ(0, pthread_attr_destroy(&attr)); if (err != 0) { fprintf(stderr, "node[%u]: pthread_create: %s\n", - GetProcessId(), strerror(err)); + uv_os_getpid(), strerror(err)); fflush(stderr); // Leave SIGUSR1 blocked. We don't install a signal handler, // receiving the signal would terminate the process. @@ -144,7 +144,7 @@ static int StartDebugSignalHandler() { DWORD pid; LPTHREAD_START_ROUTINE* handler; - pid = GetCurrentProcessId(); + pid = uv_os_getpid(); if (GetDebugSignalHandlerMappingName(pid, mapping_name, @@ -692,4 +692,3 @@ bool Agent::IsWaitingForConnect() { } // namespace inspector } // namespace node - diff --git a/src/node.cc b/src/node.cc index 13d93796e4e64c..f9b67dd675dc13 100644 --- a/src/node.cc +++ b/src/node.cc @@ -3465,7 +3465,7 @@ void SetupProcessObject(Environment* env, process->Set(FIXED_ONE_BYTE_STRING(env->isolate(), "env"), process_env); READONLY_PROPERTY(process, "pid", - Integer::New(env->isolate(), GetProcessId())); + Integer::New(env->isolate(), uv_os_getpid())); READONLY_PROPERTY(process, "features", GetFeatures(env)); CHECK(process->SetAccessor(env->context(), diff --git a/src/node_internals.h b/src/node_internals.h index edab3858f1b411..4ffa0ed930b2c9 100644 --- a/src/node_internals.h +++ b/src/node_internals.h @@ -248,7 +248,6 @@ void RegisterSignalHandler(int signal, bool reset_handler = false); #endif -uint32_t GetProcessId(); bool SafeGetenv(const char* key, std::string* text); std::string GetHumanReadableProcessName(); diff --git a/src/util.cc b/src/util.cc index 2aa9fb026ea315..77824acb03a610 100644 --- a/src/util.cc +++ b/src/util.cc @@ -22,16 +22,9 @@ #include "string_bytes.h" #include "node_buffer.h" #include "node_internals.h" +#include "uv.h" #include -#ifdef __POSIX__ -#include // getpid() -#endif - -#ifdef _MSC_VER -#include // GetCurrentProcessId() -#endif - namespace node { using v8::Isolate; @@ -122,15 +115,7 @@ std::string GetHumanReadableProcessName() { void GetHumanReadableProcessName(char (*name)[1024]) { char title[1024] = "Node.js"; uv_get_process_title(title, sizeof(title)); - snprintf(*name, sizeof(*name), "%s[%u]", title, GetProcessId()); -} - -uint32_t GetProcessId() { -#ifdef _WIN32 - return GetCurrentProcessId(); -#else - return getpid(); -#endif + snprintf(*name, sizeof(*name), "%s[%u]", title, uv_os_getpid()); } } // namespace node