delta/node-new.git/src/node_env_var.cc, branch docs-fix

src: snapshot node

2020-07-18T03:23:21+00:00

This runs `lib/internal/bootstrap/node.js` before creating
the builtin snapshot and deserialize the loaders from the
snapshot in deserialization mode.

PR-URL: https://github.com/nodejs/node/pull/32984
Reviewed-By: Anna Henningsen 
Reviewed-By: Daniel Bevenius

src,doc,test: remove String::New default parameter

2020-07-14T13:13:34+00:00

`kNormal` has been the implicit default for a while now (since V8 7.6).

Refs: https://github.com/v8/v8/commit/e0d7f816990ada28ebe1281ca9431236ef8c6e4f

PR-URL: https://github.com/nodejs/node/pull/34248
Reviewed-By: Ben Noordhuis 
Reviewed-By: Tobias Nießen 
Reviewed-By: James M Snell

src: retrieve binding data from the context

2020-05-06T04:44:04+00:00

Instead of passing them through the data bound to function
templates, store references to them in a list embedded inside
the context.
This makes the function templates more context-independent,
and makes it possible to embed binding data in non-main contexts.

Co-authored-by: Anna Henningsen 

PR-URL: https://github.com/nodejs/node/pull/33139
Reviewed-By: James M Snell 
Reviewed-By: Joyee Cheung

src: do not compare against wide characters

2020-04-27T17:31:11+00:00

PR-URL: https://github.com/nodejs/node/pull/32921
Reviewed-By: Ben Noordhuis 
Reviewed-By: David Carlier 
Reviewed-By: James M Snell 
Reviewed-By: Franziska Hinkelmann 
Reviewed-By: Zeyu Yang 
Reviewed-By: Anna Henningsen 
Reviewed-By: Juan José Arboleda

src: fix empty-named env var assertion failure

2020-04-27T17:31:09+00:00

Setting an environment variable with an empty name on Windows resulted
in an assertion failure, because it was checked for an '=' sign at the
beginning without verifying the length was greater than 0.

Fixes: https://github.com/nodejs/node/issues/32920
Refs: https://github.com/nodejs/node/pull/27310

PR-URL: https://github.com/nodejs/node/pull/32921
Reviewed-By: Ben Noordhuis 
Reviewed-By: David Carlier 
Reviewed-By: James M Snell 
Reviewed-By: Franziska Hinkelmann 
Reviewed-By: Zeyu Yang 
Reviewed-By: Anna Henningsen 
Reviewed-By: Juan José Arboleda

worker: fix process.env var empty key access

2020-04-27T17:31:06+00:00

PR-URL: https://github.com/nodejs/node/pull/32921
Reviewed-By: Ben Noordhuis 
Reviewed-By: David Carlier 
Reviewed-By: James M Snell 
Reviewed-By: Franziska Hinkelmann 
Reviewed-By: Zeyu Yang 
Reviewed-By: Anna Henningsen 
Reviewed-By: Juan José Arboleda

src: remove unnecessary 'Local.As' operation

2020-03-30T15:12:31+00:00

PR-URL: https://github.com/nodejs/node/pull/32286
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig 
Reviewed-By: Gus Caplan 
Reviewed-By: David Carlier 
Reviewed-By: Tobias Nießen 
Reviewed-By: James M Snell

src: refactor debug category parsing

2020-03-09T21:52:38+00:00

Move the debug category parsing into a new EnabledDebugList class,
so that it can be reused outside of Environment.

PR-URL: https://github.com/nodejs/node/pull/31884
Reviewed-By: Anna Henningsen

src: improve KVStore API

2020-03-09T12:08:14+00:00

This adds `const char*` based APIs to KVStore to avoid multiple string
conversions (char -> Utf8 -> Local -> char etc.) when possible.

PR-URL: https://github.com/nodejs/node/pull/31773
Reviewed-By: Anna Henningsen 
Reviewed-By: Joyee Cheung 
Reviewed-By: James M Snell

src: suppress warning in src/node_env_var.cc

2020-01-02T23:17:27+00:00

Initialise the superclass object in the copy constructor of mapKVStore

PR-URL: https://github.com/nodejs/node/pull/31136
Refs: https://github.com/nodejs/node/issues/18983
Reviewed-By: Anna Henningsen 
Reviewed-By: Rich Trott 
Reviewed-By: Luigi Pinca 
Reviewed-By: David Carlier 
Reviewed-By: Colin Ihrig