Skip to content

Commit

Permalink
doc: clarify text about process not responding
Browse files Browse the repository at this point in the history
The existing text about processes not responding is unclear, at least to
me. Suggestions for clarification welcome, but I think the best thing
might be to state that the process may stop responding and leave it at
that. The explanantion (about asynchronous listeners) is not clear to
me. (Why would the fact that the listeners are asynchronous matter?) If
it's an unnecessary detail (as seems likely), let's remove it.

PR-URL: #36117
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
  • Loading branch information
Trott authored and codebytere committed Nov 22, 2020
1 parent 3d623d8 commit 142cacd
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions doc/api/process.md
Original file line number Diff line number Diff line change
Expand Up @@ -525,10 +525,8 @@ process.on('SIGTERM', handle);
* `'SIGSTOP'` cannot have a listener installed.
* `'SIGBUS'`, `'SIGFPE'`, `'SIGSEGV'` and `'SIGILL'`, when not raised
artificially using kill(2), inherently leave the process in a state from
which it is not safe to attempt to call JS listeners. Doing so might lead to
the process hanging in an endless loop, since listeners attached using
`process.on()` are called asynchronously and therefore unable to correct the
underlying problem.
which it is not safe to call JS listeners. Doing so might cause the process
to stop responding.
* `0` can be sent to test for the existence of a process, it has no effect if
the process exists, but will throw an error if the process does not exist.

Expand Down

0 comments on commit 142cacd

Please sign in to comment.