delta/node-new.git/src/node_os.cc, branch crypto_set_integer_macro

os: add os.{get,set}Priority()

2018-08-22T21:27:16+00:00

PR-URL: https://github.com/nodejs/node/pull/22407
Reviewed-By: James M Snell 
Reviewed-By: Refael Ackermann

src: use UTF-8 for naming interfaces in unix

2018-07-29T15:04:48+00:00

Use a UTF-8 encoded string for naming interfaces in unix, instead of
using Latin-1, as had been done on windows previously.

PR-URL: https://github.com/nodejs/node/pull/21926
Reviewed-By: Anna Henningsen 
Reviewed-By: Tobias Nießen 
Reviewed-By: Anatoli Papirovski 
Reviewed-By: James M Snell

src: remove calls to deprecated v8 functions (NewFromUtf8)

2018-07-29T14:59:31+00:00

Remove all calls to deprecated v8 functions (here: String::NewFromUtf8) inside
the code (src directory only).

PR-URL: https://github.com/nodejs/node/pull/21926
Reviewed-By: Anna Henningsen 
Reviewed-By: Tobias Nießen 
Reviewed-By: Anatoli Papirovski 
Reviewed-By: James M Snell

src: use nullptr instead of NULL

2017-12-01T06:45:00+00:00

PR-URL: https://github.com/nodejs/node/pull/17373
Reviewed-By: Jon Moss 
Reviewed-By: Anna Henningsen 
Reviewed-By: Timothy Gu 
Reviewed-By: Colin Ihrig 
Reviewed-By: Michael Dawson 
Reviewed-By: Sakthipriyan Vairamani 
Reviewed-By: Tobias Nießen

src: explicitly register built-in modules

2017-11-13T23:17:34+00:00

Previously, built-in modules are registered before main() via
__attribute__((constructor)) mechanism in GCC and similiar
mechanism in MSVC. This causes some issues when node is built as
static library. Calling module registration function for built-in
modules in node::Init() helps to avoid the issues.

Signed-off-by: Yihong Wang 
PR-URL: https://github.com/nodejs/node/pull/16565
Refs: https://github.com/nodejs/node/pull/14986#issuecomment-332758206
Reviewed-By: Gireesh Punathil 
Reviewed-By: Anna Henningsen 
Reviewed-By: Ben Noordhuis 
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig

os: migrate node_os.cc to internal/errors

2017-11-02T18:58:38+00:00

PR-URL: https://github.com/nodejs/node/pull/16567
Reviewed-By: Joyee Cheung 
Reviewed-By: Michael Dawson

src: refactor `#include` handling

2017-09-14T15:38:38+00:00

`node_internals.h` already includes the most common headers,
so double includes can be avoided in a lot of cases. Also don’t include
`node_internals.h` from `node.h` implicitly anymore, as that is mostly
unnecessary.

PR-URL: https://github.com/nodejs/node/pull/14697
Reviewed-By: Tobias Nießen 
Reviewed-By: Refael Ackermann 
Reviewed-By: James M Snell

src: use proper errors as coming from StringBytes

2017-08-24T19:05:07+00:00

The previous errors were incorrect here, as the code
only failed in situations where strings exceeded size limits or
an OOM situation was encountered, not for invalid encodings
(which aren’t even detected explicitly).

Unfortunately, these situations are hard to test for.

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

src: do proper StringBytes error handling

2017-05-03T17:21:32+00:00

- Return `MaybeLocal`s from `StringBytes::Encode`
- Add an `error` out parameter to pass JS exceptions to the callers
  (instead of directly throwing)
- Simplify some of the string generation methods in `string_bytes.cc`
  by unifying the `EXTERN_APEX` logic
- Reduce usage of deprecated V8 APIs.
- Remove error handling logic from JS, the `buffer.*Slice()` methods
  now throw errors themselves.
- Left TODO comments for future semver-major error message
  improvements.

This paves the way for better error messages coming out of the
StringBytes methods.

Ref: https://github.com/nodejs/node/issues/3175
PR-URL: https://github.com/nodejs/node/pull/12765
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: James M Snell 
Reviewed-By: Tobias Nießen

os,vm: fix segfaults and CHECK failure

2017-04-14T21:02:45+00:00

Fixes multiple possible segmentation faults in node_contextify.cc and
an assertion failure in node_os.cc

Fixes: https://github.com/nodejs/node/issues/12369
Fixes: https://github.com/nodejs/node/issues/12370
PR-URL: https://github.com/nodejs/node/pull/12371
Reviewed-By: Anna Henningsen