Skip to content

Conversation

@cjihrig
Copy link
Contributor

@cjihrig cjihrig commented Mar 18, 2019

Coverity was complaining that this field was not initialized.

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

Coverity was complaining that this field was not initialized.
@nodejs-github-bot nodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Mar 18, 2019
@nodejs-github-bot
Copy link
Collaborator

@danbev
Copy link
Contributor

danbev commented Mar 21, 2019

@danbev
Copy link
Contributor

danbev commented Mar 22, 2019

Re-run of failing node-test-commit-arm-fanned (✔️)

@danbev
Copy link
Contributor

danbev commented Mar 22, 2019

Landed in 76387c1.

@danbev danbev closed this Mar 22, 2019
danbev pushed a commit that referenced this pull request Mar 22, 2019
Coverity was complaining that this field was not initialized.

PR-URL: #26739
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Yongsheng Zhang <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@cjihrig cjihrig deleted the coverity branch March 25, 2019 22:05
targos pushed a commit to targos/node that referenced this pull request Mar 27, 2019
Coverity was complaining that this field was not initialized.

PR-URL: nodejs#26739
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Yongsheng Zhang <[email protected]>
Reviewed-By: James M Snell <[email protected]>
targos pushed a commit that referenced this pull request Mar 27, 2019
Coverity was complaining that this field was not initialized.

PR-URL: #26739
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: Yongsheng Zhang <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Sign up for free to 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.

5 participants