Skip to content

src: fix formatting of PIDs - #21852

Closed
tniessen wants to merge 1 commit into
nodejs:masterfrom
tniessen:src-correctly-format-pids
Closed

src: fix formatting of PIDs#21852
tniessen wants to merge 1 commit into
nodejs:masterfrom
tniessen:src-correctly-format-pids

Conversation

@tniessen

Copy link
Copy Markdown
Member

uv_pid_t seems to be signed on all supported platforms, so %d seems appropriate.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-botnodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Jul 17, 2018
@maclover7

Copy link
Copy Markdown
Contributor

@maclover7maclover7 added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jul 20, 2018
tniessen added a commit that referenced this pull request Jul 22, 2018
PR-URL: #21852
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
@tniessen

Copy link
Copy Markdown
MemberAuthor

Landed in 8ac9163.

targos pushed a commit that referenced this pull request Jul 24, 2018
PR-URL: #21852
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
@targostargos mentioned this pull request Jul 31, 2018
@tniessentniessen removed the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Sep 12, 2018
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++Issues and PRs that require attention from people who are familiar with C++.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@tniessen@nodejs-github-bot@maclover7@jasnell@lpinca@joyeecheung@trivikr