From 00224771e32e4d051e5ea33b7e854f0031359912 Mon Sep 17 00:00:00 2001 From: Fedor Indutny Date: Wed, 28 Mar 2012 01:40:45 +0700 Subject: [PATCH] Revert "debug: Wait 50ms before running the main module" This reverts commit c781f17742170f2e127f9ee6652c56b406c30586. --- src/node.js | 29 ++++------------------------- 1 file changed, 4 insertions(+), 25 deletions(-) diff --git a/src/node.js b/src/node.js index 512a647dad8..3229416dfb4 100644 --- a/src/node.js +++ b/src/node.js @@ -89,31 +89,10 @@ } var Module = NativeModule.require('module'); - - if (global.v8debug && - process.execArgv.some(function(arg) { - return arg.match(/^--debug-brk(=[0-9]*)?$/); - })) { - - // XXX Fix this terrible hack! - // - // Give the client program a few ticks to connect. - // Otherwise, there's a race condition where `node debug foo.js` - // will not be able to connect in time to catch the first - // breakpoint message on line 1. - // - // A better fix would be to somehow get a message from the - // global.v8debug object about a connection, and runMain when - // that occurs. --isaacs - - setTimeout(Module.runMain, 50); - - } else { - // REMOVEME: nextTick should not be necessary. This hack to get - // test/simple/test-exception-handler2.js working. - // Main entry point into most programs: - process.nextTick(Module.runMain); - } + // REMOVEME: nextTick should not be necessary. This hack to get + // test/simple/test-exception-handler2.js working. + // Main entry point into most programs: + process.nextTick(Module.runMain); } else { var Module = NativeModule.require('module');