diff options
author | legendecas <legendecas@gmail.com> | 2022-08-26 00:20:07 +0800 |
---|---|---|
committer | legendecas <legendecas@gmail.com> | 2022-09-02 14:01:39 +0800 |
commit | 7f496fefb67c6a9ad22db87eb69104e9a9abb6d2 (patch) | |
tree | a43d0cc6ff19ee52ff1c27abc6d99b39f924ac96 /deps/npm/node_modules/diff/lib/patch | |
parent | d0f73d383d5e1ed58c96a272036744d4bed22b8f (diff) | |
download | node-new-7f496fefb67c6a9ad22db87eb69104e9a9abb6d2.tar.gz |
report: fix missing section javascriptHeap on OOMError
`Environment::GetCurrent` may not available in the context of OOM.
Removes the cyclic `Environment::GetCurrent` and `env->isolate()`
calls to ensure both `isolate` and `env` is present if available.
However, this behavior is not guaranteed. As
`Environment::GetCurrent` didn't allocate new handles in the heap,
when a Context is entered it can still get the valid env pointer.
Removes the unstable assertion of the absence of env in the test.
PR-URL: https://github.com/nodejs/node/pull/44398
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Diffstat (limited to 'deps/npm/node_modules/diff/lib/patch')
0 files changed, 0 insertions, 0 deletions