delta/node-new.git/src/node.cc, branch Ninja.py-StringIO

src: use non-deprecated overload of V8::SetFlagsFromString

2019-08-01T10:56:53+00:00

PR-URL: https://github.com/nodejs/node/pull/28016
Reviewed-By: Colin Ihrig 
Reviewed-By: Refael Ackermann (רפאל פלחי) 
Reviewed-By: Rich Trott 
Reviewed-By: Michael Dawson 
Reviewed-By: Jiawen Geng

src: read break_node_first_line from the inspect options

2019-07-30T04:08:25+00:00

There are cases where the debug_options() on the env are
different to the options that were passed into inspector::Agent.

PR-URL: https://github.com/nodejs/node/pull/28034
Reviewed-By: James M Snell 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Anna Henningsen

src: move relative uptime init

2019-07-28T05:51:10+00:00

PR-URL: https://github.com/nodejs/node/pull/28849
Reviewed-By: Anna Henningsen 
Reviewed-By: Rich Trott 
Reviewed-By: Colin Ihrig 
Reviewed-By: Luigi Pinca

build: remove broken intel vtune support

2019-07-14T22:25:27+00:00

Support for VTune profiling was added in commit a881b53 from November
2015 but has since bitrotted. Remove it.

Fixes: https://github.com/nodejs/node/issues/28310
Refs: https://github.com/nodejs/node/pull/3785

PR-URL: https://github.com/nodejs/node/pull/28522
Reviewed-By: Colin Ihrig 
Reviewed-By: Rich Trott 
Reviewed-By: Luigi Pinca

src: implement runtime option --no-node-snapshot for debugging

2019-07-09T17:32:25+00:00

PR-URL: https://github.com/nodejs/node/pull/28567
Refs: https://github.com/nodejs/node/issues/28558
Reviewed-By: Gus Caplan 
Reviewed-By: Anna Henningsen 
Reviewed-By: Rich Trott

src: block SIGTTOU before calling tcsetattr()

2019-07-06T18:44:29+00:00

We might be a background job that doesn't own the TTY so block SIGTTOU
before making the tcsetattr() call, otherwise that signal suspends us.

This is a better fix than PR #28490 for issue #28479.

Fixes: https://github.com/nodejs/node/issues/28530
Fixes: https://github.com/nodejs/node/issues/28479
Refs: https://github.com/nodejs/node/pull/28490

PR-URL: https://github.com/nodejs/node/pull/28535
Reviewed-By: Gireesh Punathil 
Reviewed-By: Richard Lau 
Reviewed-By: Rich Trott 
Reviewed-By: Colin Ihrig 
Reviewed-By: Sam Roberts

src: don't abort on EIO when restoring tty

2019-07-02T00:48:28+00:00

EIO has been observed to be returned by the Linux kernel under some
circumstances. Reading through drivers/tty/tty_io*.c, it seems to
indicate the tty went away. Of course none of this is documented.

Fixes: https://github.com/nodejs/node/issues/28479

PR-URL: https://github.com/nodejs/node/pull/28490
Reviewed-By: Michaël Zasso 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Colin Ihrig 
Reviewed-By: Anna Henningsen 
Reviewed-By: Gus Caplan

src: refactor uncaught exception handling

2019-06-19T08:16:37+00:00

The C++ land `node::FatalException()` is not in fact fatal anymore.
It gives the user a chance to handle the uncaught exception
globally by listening to the `uncaughtException` event. This patch
renames it to `TriggerUncaughtException` in C++ to avoid the confusion.

In addition rename the JS land handler to `onGlobalUncaughtException`
to reflect its purpose - we have to keep the alias
`process._fatalException` and use that for now since it has been
monkey-patchable in the user land.

This patch also

- Adds more comments to the global uncaught exception handling routine
- Puts a few other C++ error handling functions into the `errors`
  namespace
- Moves error-handling-related bindings to the `errors` binding.

Refs: https://github.com/nodejs/node/commit/2b252acea47af3ebeac3d7e68277f015667264cc

PR-URL: https://github.com/nodejs/node/pull/28257
Reviewed-By: Anna Henningsen 
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Franziska Hinkelmann

src: forbid reset_handler for SIGSEGV handling

2019-06-14T17:11:41+00:00

This is not easily implementable, and should be explicitly disallowed.

PR-URL: https://github.com/nodejs/node/pull/27775
Refs: https://github.com/nodejs/node/pull/27246
Reviewed-By: Ben Noordhuis 
Reviewed-By: James M Snell 
Reviewed-By: Rich Trott

src: reset SIGSEGV handler before crashing

2019-06-14T17:11:40+00:00

Without this, we would re-enter the signal handler immediately
after re-raising the signal, leading to an infinite loop.

PR-URL: https://github.com/nodejs/node/pull/27775
Refs: https://github.com/nodejs/node/pull/27246
Reviewed-By: Ben Noordhuis 
Reviewed-By: James M Snell 
Reviewed-By: Rich Trott