Skip to content

Commit

Permalink
src: make SIGPROF message a real warning
Browse files Browse the repository at this point in the history
This commit replaces a fprintf() with a call to
ProcessEmitWarning().

Refs: #12706
PR-URL: #12709
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
  • Loading branch information
cjihrig committed May 11, 2017
1 parent 9ce2271 commit b6001a2
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/signal_wrap.cc
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,8 @@ class SignalWrap : public HandleWrap {
if (signum == SIGPROF) {
Environment* env = Environment::GetCurrent(args);
if (env->inspector_agent()->IsStarted()) {
fprintf(stderr, "process.on(SIGPROF) is reserved while debugging\n");
ProcessEmitWarning(env,
"process.on(SIGPROF) is reserved while debugging");
return;
}
}
Expand Down
17 changes: 17 additions & 0 deletions test/parallel/test-warn-sigprof.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
'use strict';
const common = require('../common');

// The inspector attempts to start when Node starts. Once started, the inspector
// warns on the use of a SIGPROF listener.

common.skipIfInspectorDisabled();

if (common.isWindows) {
common.skip('test does not apply to Windows');
return;
}

common.expectWarning('Warning',
'process.on(SIGPROF) is reserved while debugging');

process.on('SIGPROF', () => {});

0 comments on commit b6001a2

Please sign in to comment.