diff options
| author | Anna Henningsen <anna@addaleax.net> | 2019-10-12 01:11:11 +0200 |
|---|---|---|
| committer | Anna Henningsen <anna@addaleax.net> | 2019-11-19 13:47:34 +0100 |
| commit | 1317ac65b4f001588b86002aa22431719683beb4 (patch) | |
| tree | de6fe6ec98b6fb8103b2e987783769fccf0930a9 /src/node_api.cc | |
| parent | a489583eda4d7cebc06516834b31dc2a4cedb1b6 (diff) | |
| download | android-node-v8-1317ac65b4f001588b86002aa22431719683beb4.tar.gz android-node-v8-1317ac65b4f001588b86002aa22431719683beb4.tar.bz2 android-node-v8-1317ac65b4f001588b86002aa22431719683beb4.zip | |
src: remove HandleWrap instances from list once closed
This allows keeping `BaseObjectPtr`s to `HandleWrap` instances.
Previously, the pointer kept the `HandleWrap` object alive, leaving
the Environment cleanup code that waits for the handle list to drain
in a busy loop, because only the `HandleWrap` destructor removed
the item from the list.
Refs: https://github.com/nodejs/quic/pull/165
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
PR-URL: https://github.com/nodejs/node/pull/30374
Refs: https://github.com/nodejs/quic/pull/141
Refs: https://github.com/nodejs/quic/pull/149
Refs: https://github.com/nodejs/quic/pull/141
Reviewed-By: David Carlier <devnexen@gmail.com>
Diffstat (limited to 'src/node_api.cc')
0 files changed, 0 insertions, 0 deletions