delta/node-new.git/src/node_internals.h, branch crypto_set_integer_macro

src: replace NO_RETURN with [[noreturn]]

2018-10-11T23:53:20+00:00

PR-URL: https://github.com/nodejs/node/pull/23337
Reviewed-By: Anna Henningsen 
Reviewed-By: Richard Lau 
Reviewed-By: Colin Ihrig 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Sakthipriyan Vairamani 
Reviewed-By: James M Snell 
Reviewed-By: Joyee Cheung 
Reviewed-By: Ben Noordhuis

src: define zlib constants in node_zlib.cc

2018-09-25T03:49:49+00:00

This is in order to avoid having two separate definitions
for `node_zlib_mode`.

PR-URL: https://github.com/nodejs/node/pull/23019
Reviewed-By: James M Snell 
Reviewed-By: Gus Caplan

src: deprecate global COUNTER_* and remove perfctr

2018-09-19T09:07:01+00:00

To support Performance Counters on Windows, a number of
global `COUNTER_` methods were added that are undocumented
and really only intended to be used internally by Node.js.

Unfortunately, the perfctr support apparently hasn't even
worked for quite a while and no one has even complained.

This removes the perfctr support and replaces the global
functions with deprecated non-ops for now, with the intent
of removing those outright in the next major release cycle.

PR-URL: https://github.com/nodejs/node/pull/22485
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Colin Ihrig 
Reviewed-By: João Reis

src: refactor `Environment::GetCurrent()` usage

2018-09-17T15:20:27+00:00

Make `Environment::GetCurrent()` return `nullptr` if the current
`Context` is not a Node.js context, and for the relevant usage of
this function, either:

- Switch to the better `GetCurrent(args)` variant
- Turn functions in to no-ops where it makes sense
- Make it a `CHECK`, i.e. an API requirement, where it make sense
- Leave a `TODO` comment for verifying what, if anything, is to be done

PR-URL: https://github.com/nodejs/node/pull/22819
Reviewed-By: Gus Caplan 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Eugene Ostroukhov 
Reviewed-By: Refael Ackermann

src: move DebugPortGetter/Setter to node_process.cc

2018-09-14T17:02:45+00:00

PR-URL: https://github.com/nodejs/node/pull/22758
Reviewed-By: Anna Henningsen 
Reviewed-By: Gus Caplan 
Reviewed-By: Refael Ackermann

src: move getActiveResources/Handles to node_process.cc

2018-09-14T17:02:44+00:00

PR-URL: https://github.com/nodejs/node/pull/22758
Reviewed-By: Anna Henningsen 
Reviewed-By: Gus Caplan 
Reviewed-By: Refael Ackermann

process: generate list of allowed env flags programmatically

2018-09-09T14:20:28+00:00

Avoids having a separate, second source of truth on this matter.

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

src: fix a typo in the comment

2018-09-03T08:19:15+00:00

PR-URL: https://github.com/nodejs/node/pull/22640
Reviewed-By: Luigi Pinca 
Reviewed-By: Michaël Zasso 
Reviewed-By: Colin Ihrig 
Reviewed-By: Richard Lau 
Reviewed-By: Ruben Bridgewater

src: make CLI options programatically accesible

2018-08-31T22:13:20+00:00

Provide `internalBinding('options')` with some utilities
around making the options parser and current options values
programatically accessible.

PR-URL: https://github.com/nodejs/node/pull/22490
Reviewed-By: Michaël Zasso

process: add allowedNodeEnvironmentFlags property

2018-08-25T01:15:19+00:00

`process.allowedNodeEnvironmentFlags` provides an API to validate and
list flags as specified in `NODE_OPTIONS` from user code.

Refs: https://github.com/nodejs/node/issues/17740
Signed-off-by: Christopher Hiller 

PR-URL: https://github.com/nodejs/node/pull/19335
Reviewed-By: Ben Noordhuis 
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Michael Dawson 
Reviewed-By: Matteo Collina 
Reviewed-By: Gibson Fahnestock 
Reviewed-By: John-David Dalton 
Reviewed-By: Sam Ruby 
Reviewed-By: Anna Henningsen