mirror of https://github.com/lukechilds/node.git
Browse Source
This patch now reports the proper throw call site for exceptions triggered within process.nextTick. So instead of this: node.js:201 throw e; // process.nextTick error, or 'error' event on first tick ^ You will now see: mydir/myscript.js:15 throw new Error('My Error'); ^ From my testing this patch causes no performance regressions, but does greatly simplify processing the nextTickQueue.v0.8.7-release
Felix Geisendörfer
13 years ago
committed by
isaacs
6 changed files with 28 additions and 32 deletions
@ -1,6 +1,6 @@ |
|||
before |
|||
|
|||
node.js:* |
|||
throw e; // process.nextTick error, or 'error' event on first tick |
|||
^ |
|||
module.js:311 |
|||
throw err; |
|||
^ |
|||
RangeError: Maximum call stack size exceeded |
|||
|
@ -1,6 +1,6 @@ |
|||
before |
|||
|
|||
node.js:* |
|||
throw e; // process.nextTick error, or 'error' event on first tick |
|||
^ |
|||
module.js:311 |
|||
throw err; |
|||
^ |
|||
MyCustomError: This is a custom message |
|||
|
@ -1,6 +1,6 @@ |
|||
before |
|||
|
|||
node.js:* |
|||
throw e; // process.nextTick error, or 'error' event on first tick |
|||
^ |
|||
module.js:311 |
|||
throw err; |
|||
^ |
|||
[object Object] |
|||
|
Loading…
Reference in new issue