delta/node-new.git/src/node_metadata.cc, branch try_Python3_on_macOS

src: clang build warning fix

2019-07-12T06:20:24+00:00

fix UB with string concatenations. += operator makes things
clearer for compiler's perspective.

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

src: move arch, platform and release into node_metadata.cc

2019-01-06T02:44:57+00:00

Move definitions of more metadata into node_metadata{.h, .cc}
so the data can be reused and easily inspected in C++.

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

zlib: add brotli support

2019-01-05T20:36:46+00:00

Refs: https://github.com/nodejs/node/pull/20458

Co-authored-by: Hackzzila 

PR-URL: https://github.com/nodejs/node/pull/24938
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Matteo Collina 
Reviewed-By: Jan Krems 
Reviewed-By: James M Snell 
Reviewed-By: Myles Borins 
Reviewed-By: Ali Ijaz Sheikh 
Reviewed-By: Daniel Bevenius

src: initialize ICU version in per_process::metadata.versions

2018-12-20T23:46:47+00:00

Instead of

- Initialize the ICU versions in JS land after consulting
  internalBinding('config').hasIntl
- Joining the version keys in C++
- Splitting the keys in JS and call into C++ again to get the value for
  each of the keys

Do:

- Guard the initialization code behind `NODE_HAVE_I18N_SUPPORT`
- Do the initialization in C++ right after ICU data is loaded
- Initialize each version directly using ICU functions/constants,
  and put them in per_process::metadata.versions. These will be
  copied into `process.versions` naturally later.
  This way, the initialization of the versions won't be called
  in worker threads again.

PR-URL: https://github.com/nodejs/node/pull/25115
Reviewed-By: Steven R Loomis 
Reviewed-By: Richard Lau 
Reviewed-By: Minwoo Jung 
Reviewed-By: James M Snell

src: move the declaration of http parser versions into node_metadata.h

2018-12-20T23:46:30+00:00

Instead of putting them in node_internals.h.

PR-URL: https://github.com/nodejs/node/pull/25115
Reviewed-By: Steven R Loomis 
Reviewed-By: Richard Lau 
Reviewed-By: Minwoo Jung 
Reviewed-By: James M Snell

src: move GetOpenSSLVersion into node_metadata.cc

2018-12-20T23:46:28+00:00

Instead of implementing it in node_crypto.cc even though the only
place that needs it is the `Metadata::Versions` constructor.

PR-URL: https://github.com/nodejs/node/pull/25115
Reviewed-By: Steven R Loomis 
Reviewed-By: Richard Lau 
Reviewed-By: Minwoo Jung 
Reviewed-By: James M Snell

src: include node_internals.h in node_metadata.cc

2018-12-13T04:38:00+00:00

Currently, if configured --without-ssl the following compiler error will
be generated:
../src/node_metadata.cc:29:12:
error: use of undeclared identifier 'llhttp_version'
  llhttp = llhttp_version;
           ^
../src/node_metadata.cc:30:17:
error: use of undeclared identifier 'http_parser_version'
  http_parser = http_parser_version;

This commit includes the node_internals.h header so that
llhttp_version and http_parser_versions are always available.

PR-URL: https://github.com/nodejs/node/pull/24933
Reviewed-By: Ben Noordhuis 
Reviewed-By: Richard Lau 
Reviewed-By: Joyee Cheung 
Reviewed-By: Colin Ihrig 
Reviewed-By: Anna Henningsen

http: make parser choice a runtime flag

2018-12-06T04:21:36+00:00

Add a `--http-parser=llhttp` vs `--http-parser=traditional`
command line switch, to make testing and comparing the new
llhttp-based implementation easier.

PR-URL: https://github.com/nodejs/node/pull/24739
Refs: https://github.com/nodejs/node/issues/24730
Reviewed-By: Joyee Cheung 
Reviewed-By: Colin Ihrig 
Reviewed-By: Fedor Indutny 
Reviewed-By: Gus Caplan 
Reviewed-By: Matheus Marchini 
Reviewed-By: Matteo Collina 
Reviewed-By: Ali Ijaz Sheikh

src: move version metadata into node_metadata{.h, .cc}

2018-12-04T02:16:39+00:00

This patch moves the computation of version metadata from node.cc
into node_metadata{.h, .cc}, and creates a macro that can be
used to iterate over the available version keys (v8, uv, .etc).
This makes the code clearer as now we no longer need to add
all the headers in node.cc just to compute the versions, and
makes it easier to reuse the version definitions.

PR-URL: https://github.com/nodejs/node/pull/24774
Reviewed-By: Richard Lau 
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig