Skip to content

Conversation

@mhdawson
Copy link
Member

Add missing initialization flagged by Coverity

Signed-off-by: Michael Dawson [email protected]

@nodejs-github-botnodejs-github-bot added inspector Issues and PRs related to the V8 inspector protocol needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Nov 29, 2021
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@Trott
Copy link
Member

Trott commented Dec 3, 2021

I forget but I think this might need a rebase to pull in a fix for the containered job that's failing.

Add missing initialization flagged by Coverity Signed-off-by: Michael Dawson <[email protected]>
@mhdawson
Copy link
MemberAuthor

Rebased

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

mhdawson added a commit that referenced this pull request Dec 10, 2021
Add missing initialization flagged by Coverity Signed-off-by: Michael Dawson <[email protected]> PR-URL: #41022 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
@mhdawson
Copy link
MemberAuthor

Landed in f32649a

danielleadams pushed a commit that referenced this pull request Dec 13, 2021
Add missing initialization flagged by Coverity Signed-off-by: Michael Dawson <[email protected]> PR-URL: #41022 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
danielleadams pushed a commit that referenced this pull request Dec 14, 2021
Add missing initialization flagged by Coverity Signed-off-by: Michael Dawson <[email protected]> PR-URL: #41022 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
danielleadams pushed a commit that referenced this pull request Jan 31, 2022
Add missing initialization flagged by Coverity Signed-off-by: Michael Dawson <[email protected]> PR-URL: #41022 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
danielleadams pushed a commit that referenced this pull request Jan 31, 2022
Add missing initialization flagged by Coverity Signed-off-by: Michael Dawson <[email protected]> PR-URL: #41022 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
Linkgoron pushed a commit to Linkgoron/node that referenced this pull request Jan 31, 2022
Add missing initialization flagged by Coverity Signed-off-by: Michael Dawson <[email protected]> PR-URL: nodejs#41022 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
Add missing initialization flagged by Coverity Signed-off-by: Michael Dawson <[email protected]> PR-URL: #41022 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
@danielleadamsdanielleadams mentioned this pull request Feb 1, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

inspectorIssues and PRs related to the V8 inspector protocolneeds-ciPRs that need a full CI run.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@mhdawson@nodejs-github-bot@Trott@jasnell@lpinca@richardlau