Skip to content

src: use GetCurrentProcessId() for process.pid

Rodrigo Muino Tomonari requested to merge github/fork/bnoordhuis/process-pid-windows into master

Commit a9c0c65f ("src: define getpid() based on OS") made src/env.cc use GetCurrentProcessId() on Windows for the PID in log messages. GetCurrentProcessId() is also what is used by libuv, OpenSSL and V8.

This commit makes process.pid use GetCurrentProcessId() instead of _getpid() for consistency.

R=@cjihrig?

CI: https://ci.nodejs.org/job/node-test-pull-request/930/

Merge request reports