mirror of
https://github.com/nodejs/node.git
synced 2024-11-30 23:43:09 +01:00
eb9748d222
This adds missing async_hooks destroy calls for sockets (in _http_agent.js) and HTTP parsers. We need to emit a destroy in AsyncWrap#AsyncReset before assigning a new async_id when the instance has already been in use and is being recycled, because in that case, we have already emitted an init for the "old" async_id. This also removes a duplicated init call for HTTP parser: Each time a new parser was created, AsyncReset was being called via the C++ Parser class constructor (super constructor AsyncWrap) and also via Parser::Reinitialize. PR-URL: https://github.com/nodejs/node/pull/23272 Fixes: https://github.com/nodejs/node/issues/19859 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com>
40 lines
653 B
JavaScript
40 lines
653 B
JavaScript
'use strict';
|
|
|
|
const is_reused_symbol = Symbol('isReused');
|
|
|
|
class FreeList {
|
|
constructor(name, max, ctor) {
|
|
this.name = name;
|
|
this.ctor = ctor;
|
|
this.max = max;
|
|
this.list = [];
|
|
}
|
|
|
|
alloc() {
|
|
let item;
|
|
if (this.list.length > 0) {
|
|
item = this.list.pop();
|
|
item[is_reused_symbol] = true;
|
|
} else {
|
|
item = this.ctor.apply(this, arguments);
|
|
item[is_reused_symbol] = false;
|
|
}
|
|
return item;
|
|
}
|
|
|
|
free(obj) {
|
|
if (this.list.length < this.max) {
|
|
this.list.push(obj);
|
|
return true;
|
|
}
|
|
return false;
|
|
}
|
|
}
|
|
|
|
module.exports = {
|
|
FreeList,
|
|
symbols: {
|
|
is_reused_symbol
|
|
}
|
|
};
|