Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 1681420

Browse files
committedJun 5, 2022
src: remove a stale comment in async_hooks
This removes a comment relevant to runtime checks for `async_hooks`. Even if `async_hooks` is experimental, the check pointed by the comment is performed as default unless either `--no-force-async-hooks-checks` is given from CLI arguments or any async_hook is enabled. Refs: nodejs#16318 Refs: nodejs#15454 (comment) Signed-off-by: Daeyeon Jeong [email protected]
1 parent 45d7ca9 commit 1681420

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed
 

‎src/env-inl.h

-2
Original file line numberDiff line numberDiff line change
@@ -179,8 +179,6 @@ inline bool AsyncHooks::pop_async_context(double async_id) {
179179

180180
// Ask for the async_id to be restored as a check that the stack
181181
// hasn't been corrupted.
182-
// Since async_hooks is experimental, do only perform the check
183-
// when async_hooks is enabled.
184182
if (UNLIKELY(fields_[kCheck] > 0 &&
185183
async_id_fields_[kExecutionAsyncId] != async_id)) {
186184
FailWithCorruptedAsyncStack(async_id);

0 commit comments

Comments
 (0)
Please sign in to comment.