Fix inconsistent options.noHandleSIGINT for windows

flatiron · Mar 9, 2020 · 756fa65 · 756fa65
1 parent 8d5495c
commit 756fa65
Showing 1 changed file with 16 additions and 14 deletions.
diff --git a/lib/prompt.js b/lib/prompt.js
@@ -86,22 +86,24 @@ prompt.start = function (options) {
   prompt.delimiter  = options.delimiter  || prompt.delimiter;
   prompt.colors     = options.colors     || prompt.colors;
 
-  if (process.platform !== 'win32' && !options.noHandleSIGINT) {
-    // windows falls apart trying to deal with SIGINT
-    process.on('SIGINT', function () {
-      stdout.write('\n');
-      process.exit(1);
-    });
-  } else {
-    // listen for the "Ctrl+C" key combination and trigger process event.
-    // See https://stackoverflow.com/questions/10021373/what-is-the-windows-equivalent-of-process-onsigint-in-node-js
-    stdin.on('keypress', function(char, key) {
-      if (key && key.ctrl && key.name == 'c') {
+  if (!options.noHandleSIGINT) {
+    if (process.platform !== 'win32') {
+      // windows falls apart trying to deal with SIGINT
+      process.on('SIGINT', function () {
         stdout.write('\n');
-        process.emit("SIGINT");
         process.exit(1);
-      }
-    });
+      });
+    } else {
+      // listen for the "Ctrl+C" key combination and trigger process event.
+      // See https://stackoverflow.com/questions/10021373/what-is-the-windows-equivalent-of-process-onsigint-in-node-js
+      stdin.on('keypress', function(char, key) {
+        if (key && key.ctrl && key.name == 'c') {
+          stdout.write('\n');
+          process.emit("SIGINT");
+          process.exit(1);
+        }
+      });
+    }
   }
 
   prompt.emit('start');