Home Original page

test: fix issues reported by Coverity by eugeneo · Pull Request #8870 · nodejs/node

@nodejs-github-bot added the test

Issues and PRs related to the tests.

label

Sep 30, 2016

@eugeneo eugeneo changed the title test: fix issues reported by Coverty test: fix issues reported by Coverity

Sep 30, 2016

@eugeneo eugeneo added c++

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

inspector

Issues and PRs related to the V8 inspector protocol

labels

Sep 30, 2016

bnoordhuis

ofrobots

bnoordhuis

eugeneo pushed a commit that referenced this pull request

Oct 13, 2016 
Wrapped the timer into class to ensure it is cleaned up properly.

PR-URL: #8870
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

jasnell pushed a commit that referenced this pull request

Oct 14, 2016 
Wrapped the timer into class to ensure it is cleaned up properly.

PR-URL: #8870
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>

MylesBorins pushed a commit that referenced this pull request

Nov 11, 2016 
Wrapped the timer into class to ensure it is cleaned up properly.

PR-URL: #8870
Reviewed-By: Ali Ijaz Sheikh <ofrobots@google.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>