delta/node-new.git/src, branch Remove-Python-3-tests

src: do not crash when accessing empty WeakRefs

2019-09-02T06:10:54+00:00

Making `.incRef()` and `.decRef()` fail silently leads to better error
messages when trying to access the underlying value (as opposed to
crashing inside these methods).

Refs: https://github.com/nodejs/node/pull/25461#issuecomment-524481482

PR-URL: https://github.com/nodejs/node/pull/29289
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Gus Caplan

src: turn `GET_OFFSET()` into an inline function

2019-09-02T05:57:06+00:00

There’s no need for this to be a macro.

PR-URL: https://github.com/nodejs/node/pull/29357
Reviewed-By: Jiawen Geng 
Reviewed-By: Masashi Hirano 
Reviewed-By: Colin Ihrig 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
Reviewed-By: Richard Lau 
Reviewed-By: Michael Dawson

src: inline `SLICE_START_END()` in node_buffer.cc

2019-09-02T05:56:56+00:00

This macro is only used once, so it doesn’t need to be a macro.

PR-URL: https://github.com/nodejs/node/pull/29357
Reviewed-By: Jiawen Geng 
Reviewed-By: Masashi Hirano 
Reviewed-By: Colin Ihrig 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
Reviewed-By: Richard Lau 
Reviewed-By: Michael Dawson

crypto: don't expose openssl internals

2019-08-29T04:42:58+00:00

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

buffer: consolidate encoding parsing

2019-08-28T19:13:38+00:00

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

src: allow --interpreted-frames-native-stack in NODE_OPTIONS

2019-08-27T22:59:06+00:00

PR-URL: https://github.com/nodejs/node/pull/27744
Reviewed-By: Richard Lau 
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig 
Reviewed-By: Rich Trott 
Reviewed-By: James M Snell 
Reviewed-By: Ruben Bridgewater

src: expose MaybeInitializeContext to allow existing contexts

2019-08-27T01:24:10+00:00

Splits the node.js specific tweak intialization of NewContext into a new
helper MaybeInitializeContext so that embedders with existing contexts
can still use them in a Node.js Environment now that primordials are
initialized and required so early.

Update MaybeInitializeContext to return MaybeLocal,

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

src: remove unused using declarations

2019-08-23T22:13:57+00:00

This commit removes unused using declarations in
src/node_contextify.cc.

PR-URL: https://github.com/nodejs/node/pull/29222
Reviewed-By: Jiawen Geng 
Reviewed-By: Anna Henningsen 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Colin Ihrig 
Reviewed-By: James M Snell 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Luigi Pinca 
Reviewed-By: Trivikram Kamat

net: allow reading data into a static buffer

2019-08-23T21:05:52+00:00

Co-Authored-By: Anna Henningsen 

PR-URL: https://github.com/nodejs/node/pull/25436
Reviewed-By: James M Snell 
Reviewed-By: Matteo Collina 
Reviewed-By: Ben Noordhuis

module: avoid passing unnecessary loop reference

2019-08-23T06:03:16+00:00

It's a sync request, which doesn't require the loop parameter to be set.

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