src: make SIGPROF message a real warning

This commit replaces a fprintf() with a call to ProcessEmitWarning(). Refs: nodejs#12706 PR-URL: nodejs#12709 Reviewed-By: Jeremiah Senkpiel <[email protected]> Reviewed-By: Sam Roberts <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
anchnk · May 19, 2017 · 18cf5fe · 18cf5fe
1 parent 47c4d96
commit 18cf5fe
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 1 deletion.
diff --git a/src/signal_wrap.cc b/src/signal_wrap.cc
@@ -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;
       }
     }

diff --git a/test/parallel/test-warn-sigprof.js b/test/parallel/test-warn-sigprof.js
@@ -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', () => {});