-
Notifications
You must be signed in to change notification settings - Fork 29.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
src: use internal/errors for startSigintWatchdog #16546
Conversation
Ping @nodejs/tsc |
doc/api/errors.md
Outdated
<a id="ERR_REPL_STARTSIGINTWATCHDOG_FAILED"></a> | ||
### ERR_REPL_STARTSIGINTWATCHDOG_FAILED | ||
|
||
The `SigintWatchdog` is an internal utility used by the `repl` module to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it too much detail? AFAICT this is triggered by a failed pthread_create
, may be we don't have to mention the watch dog thing, simply saying that node is unable to monitor SIGINT
is enough? (ERR_REPL_SIGINTWATCH_FAILED
)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
EDIT: paste error, s/sigmask/pthread_create/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree, both the error code & the docs on it do have too much detail – ERR_CANNOT_WATCH_SIGINT
or so should be fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Move the throw out of c++ and into js using internal/errors
f99ee60
to
0d89d4d
Compare
ping @joyeecheung and @addaleax |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Move the throw out of c++ and into js using internal/errors PR-URL: #16546 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
Landed in 67c8511 |
Move the throw out of c++ and into js using internal/errors
Checklist
make -j4 test
(UNIX), orvcbuild test
(Windows) passesAffected core subsystem(s)
src, repl