delta/node-new.git/src, branch main

deps: update undici to 5.22.1

2023-05-16T16:36:34+00:00

PR-URL: https://github.com/nodejs/node/pull/47994
Reviewed-By: Matthew Aitken 
Reviewed-By: Moshe Atlow 
Reviewed-By: Mohammed Keyvanzadeh 
Reviewed-By: Mestery

doc: fix typo in binding functions

2023-05-16T14:26:17+00:00

According to usage(node::util::RegisterExternalReferences) of
below line, namespace has to be `util`(not `utils`).

PR-URL: https://github.com/nodejs/node/pull/48003
Reviewed-By: Darshan Sen 
Reviewed-By: Daeyeon Jeong 
Reviewed-By: Harshitha K P 
Reviewed-By: Luigi Pinca

permission: remove unused function declaration

2023-05-15T14:18:07+00:00

RestrictAccess() is declared, but it's not used.

PR-URL: https://github.com/nodejs/node/pull/47957
Reviewed-By: Tobias Nießen 
Reviewed-By: Colin Ihrig 
Reviewed-By: Yagiz Nizipli 
Reviewed-By: Rafael Gonzaga 
Reviewed-By: Darshan Sen 
Reviewed-By: Luigi Pinca 
Reviewed-By: Khaidi Chu

src: deduplicate X509Certificate::Fingerprint*

2023-05-15T12:49:00+00:00

All three functions do the same, except using different cryptographic
hash functions. Move the common logic into a new template and use it
directly.

PR-URL: https://github.com/nodejs/node/pull/47978
Reviewed-By: Ben Noordhuis 
Reviewed-By: Filip Skokan

vm: fix crash when setting proto on context's globalThis

2023-05-15T09:24:15+00:00

PR-URL: https://github.com/nodejs/node/pull/47939
Reviewed-By: Zeyu "Alex" Yang 
Reviewed-By: James M Snell 
Reviewed-By: Darshan Sen 
Reviewed-By: Anna Henningsen

src: stop copying code cache, part 2

2023-05-15T04:37:24+00:00

This removes more copies of the code cache data.

First: for the builtin snapshot, we were copying the code cache to
create a `std::vector`. This was slowing down static
intialization. Change it to use a good old `uint8_t*` and `size_t`
rather than a vector. For the case of embedder provided snapshots, we
also add an `owning_ptr` so that we can properly cleanup owned values
created from the snapshot.

Second: whenever the code cache was hit, we would remove the bytecode
from the code cache, and then reserialize it from the compiled function.
This was pretty slow. Change the code so that we can reuse the same code
cache multiple times. If the code cache is rejected (say, because the
user added V8 options), then we need to generate the bytecode, in which
case we again use `owning_ptr` to ensure that the underlying code cache
is freed.

Combined, these changes improve the misc/startup.js benchmarks
significantly (p < 0.001):

* process,benchmark/fixtures/require-builtins: 22.15%
* process,test/fixtures/semicolon: 8.55%
* worker,benchmark/fixtures/require-builtins: 26.52%
* worker,test/fixtures/semicolon: 21.52%

PR-URL: https://github.com/nodejs/node/pull/47958
Reviewed-By: Joyee Cheung 
Reviewed-By: Minwoo Jung

src: add cjs_module_lexer_version base64_version

2023-05-14T12:16:47+00:00

src: add cjs_module_lexer_version base64_version
PR-URL: https://github.com/nodejs/node/pull/45629
Refs: https://github.com/nodejs/node/issues/45260
Reviewed-By: James M Snell 
Reviewed-By: Minwoo Jung 
Reviewed-By: Darshan Sen 
Reviewed-By: Antoine du Hamel

src: move BlobSerializerDeserializer to a separate header file

2023-05-11T12:49:24+00:00

This should make it possible to reuse the BlobSerializer and the
BlobDeserializer classes in SEAs to generate and parse the injected
blob.

This change also resolves this TODO:
https://github.com/nodejs/node/blob/4f69aae6a04a460f267005dcf6551959064b3238/src/node_snapshotable.cc#L187

Refs: https://github.com/nodejs/node/pull/47458
Signed-off-by: Darshan Sen 
PR-URL: https://github.com/nodejs/node/pull/47933
Reviewed-By: Joyee Cheung 
Reviewed-By: Colin Ihrig

dns: call `ada::idna::to_ascii` directly from c++

2023-05-10T16:04:35+00:00

PR-URL: https://github.com/nodejs/node/pull/47920
Fixes: https://github.com/nodejs/performance/issues/77
Reviewed-By: James M Snell 
Reviewed-By: Matteo Collina

src: rename SKIP_CHECK_SIZE to SKIP_CHECK_STRLEN

2023-05-10T13:20:04+00:00

SKIP_CHECK_VALUE is a string literal, so its size is the length of the
string in chars plus one. The buffer buf is also always null-terminated,
so its size should match the size of SKIP_CHECK_VALUE, which is _not_
SKIP_CHECK_SIZE.

Rename SKIP_CHECK_SIZE to be consistent with C/C++ terminology.

PR-URL: https://github.com/nodejs/node/pull/47845
Reviewed-By: James M Snell 
Reviewed-By: Darshan Sen 
Reviewed-By: Shelley Vohr