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

src: remove memory_tracker-inl.h from header files

2019-05-23T15:51:39+00:00

Inline headers should only be included into the .cc files that use them.

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

src: declare unused priv argument

2019-05-13T20:43:17+00:00

PR-URL: https://github.com/nodejs/node/pull/27631
Reviewed-By: Joyee Cheung 
Reviewed-By: James M Snell 
Reviewed-By: Rich Trott 
Reviewed-By: Colin Ihrig

build: improve embedded code-cache detection

2019-04-22T18:48:50+00:00

PR-URL: https://github.com/nodejs/node/pull/27311
Reviewed-By: Joyee Cheung 
Reviewed-By: Richard Lau

lib: move DTRACE_* probes out of global scope

2019-03-12T14:19:36+00:00

The DTRACE_* probes have been global for no really good reason.
Move those into an internalBinding.

PR-URL: https://github.com/nodejs/node/pull/26541
Reviewed-By: Anna Henningsen 
Reviewed-By: Michaël Zasso 
Reviewed-By: Matteo Collina 
Reviewed-By: Matheus Marchini 
Reviewed-By: Gus Caplan 
Reviewed-By: Ruben Bridgewater

src: use the config binding to carry --no-browser-globals

2019-03-06T23:27:40+00:00

Instead of setting it in the process object, since this is
a configure-time option. Also added a shim that can be
deprecated and removed some time later.

PR-URL: https://github.com/nodejs/node/pull/26228
Reviewed-By: Daniel Bevenius 
Reviewed-By: James M Snell 
Reviewed-By: Ruben Bridgewater

src: fix warnings around node_options

2019-03-04T01:03:31+00:00

* header explicit usage, order, and reduce use of `*-inl.h`
* pointer -> const reference when possible
* no variable recyclicng
* `std::begin/end` prefered over `instance.begin/end`
* `USE` for explicit unused resaults

PR-URL: https://github.com/nodejs/node/pull/26280
Fixes: https://github.com/nodejs/node/issues/25593
Reviewed-By: Joyee Cheung

src: remove `process.binding('config').fipsForced`

2019-02-20T15:42:11+00:00

Instead use `require('internal/options').getOptionValue` to
query to value of `--force-fips`.

PR-URL: https://github.com/nodejs/node/pull/26178
Reviewed-By: Daniel Bevenius 
Reviewed-By: Anna Henningsen 
Reviewed-By: Richard Lau 
Reviewed-By: James M Snell

src: remove `process.binding('config').debugOptions`

2019-02-19T08:38:20+00:00

`process.binding('config').debugOptions`, which contains the initial
values of parsed debugger-related CLI options, has been used for
internal testing. This patch removes them and uses `internal/options`
to query the values in the tests instead.

PR-URL: https://github.com/nodejs/node/pull/25999
Reviewed-By: Daniel Bevenius 
Reviewed-By: Anna Henningsen 
Reviewed-By: James M Snell

lib: use `internal/options` to query `--abort-on-uncaught-exception`

2019-02-08T00:09:56+00:00

Instead of using
`internalBinding('config').shouldAbortOnUncaughtException`.
Also removes that property from the binding.

PR-URL: https://github.com/nodejs/node/pull/25862
Reviewed-By: Richard Lau 
Reviewed-By: Luigi Pinca 
Reviewed-By: Gus Caplan 
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig

process: expose process.features.inspector

2019-02-01T21:41:45+00:00

Instead of using process.config.variables.v8_enable_inspector
to detect whether inspector is enabled in the build.

PR-URL: https://github.com/nodejs/node/pull/25819
Refs: https://github.com/nodejs/node/issues/25343
Reviewed-By: Eugene Ostroukhov 
Reviewed-By: Gus Caplan 
Reviewed-By: James M Snell 
Reviewed-By: Richard Lau 
Reviewed-By: Colin Ihrig 
Reviewed-By: Anna Henningsen