src: use GetCurrentProcessId() for process.pid · nodejs/node@9e9346f · GitHub
Skip to content

Commit 9e9346f

Browse files
bnoordhuisrvagg
authored andcommitted
src: use GetCurrentProcessId() for process.pid
Commit a9c0c65 ("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. PR-URL: #4163 Reviewed-By: Brian White <mscdex@mscdex.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Rod Vagg <rod@vagg.org>
1 parent da6be4d commit 9e9346f

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

src/node.cc

Lines changed: 1 addition & 2 deletions

0 commit comments

Comments
 (0)