- Apr 20, 2020
-
-
cjihrig authored
Notable changes: - gyp support has been removed. - recvmmsg messages are returned in the correct order. - IBMi cmake support has been added. - uv_fs_lutime() has been added. - uv_fs_statfs() on Windows properly handles file paths. PR-URL: https://github.com/nodejs/node/pull/32866 Reviewed-By:
James M Snell <jasnell@gmail.com> Reviewed-By:
Richard Lau <riclau@uk.ibm.com> Reviewed-By:
Anna Henningsen <anna@addaleax.net> Reviewed-By:
David Carlier <devnexen@gmail.com> Reviewed-By:
Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By:
Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By:
Jiawen Geng <technicalcute@gmail.com> Reviewed-By:
Juan José Arboleda <soyjuanarbol@gmail.com>
-
Rich Trott authored
Refs: https://github.com/nodejs/nodejs.org/pull/3106#issuecomment-614258785 PR-URL: https://github.com/nodejs/node/pull/32903 Reviewed-By:
Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By:
Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By:
Luigi Pinca <luigipinca@gmail.com>
-
Gabriel Schulhof authored
We introduce status `napi_would_deadlock` to be used as a return status by `napi_call_threadsafe_function` if the call is made with `napi_tsfn_blocking` on the main thread and the queue is full. Fixes: https://github.com/nodejs/node/issues/32615 Signed-off-by:
Gabriel Schulhof <gabriel.schulhof@intel.com> PR-URL: https://github.com/nodejs/node/pull/32860 Reviewed-By:
Michael Dawson <michael_dawson@ca.ibm.com> Reviewed-By:
Zeyu Yang <himself65@outlook.com>
-
- Apr 19, 2020
-
-
Juan José Arboleda authored
Based on https://github.com/nodejs/help/issues/2600#issuecomment-612857153 the condition (amount < 0) won't be possible. PR-URL: https://github.com/nodejs/node/pull/32818 Reviewed-By:
Yongsheng Zhang <zyszys98@gmail.com> Reviewed-By:
Shelley Vohr <codebytere@gmail.com> Reviewed-By:
Colin Ihrig <cjihrig@gmail.com> Reviewed-By:
Gus Caplan <me@gus.host>
-
Jesus Hernandez authored
PR-URL: https://github.com/nodejs/node/pull/32662 Reviewed-By:
-
David Daza authored
The if statement inside the _writeHostObject function returns an expression which makes the else block unnecessary. PR-URL: https://github.com/nodejs/node/pull/32644 Reviewed-By:
-
Vadzim Zieńka authored
Call iterator.return() if not all of its values are consumed. Fixes: https://github.com/nodejs/node/issues/32842 PR-URL: https://github.com/nodejs/node/pull/32844 Reviewed-By:
Robert Nagy <ronagy@icloud.com> Reviewed-By:
Matteo Collina <matteo.collina@gmail.com> Reviewed-By:
-
Robert Nagy authored
PR-URL: https://github.com/nodejs/node/pull/32886 Reviewed-By:
-
Robert Nagy authored
Simplifies Writable.end() by inlining and de-duplicating code. PR-URL: https://github.com/nodejs/node/pull/32882 Reviewed-By:
Gerhard Stöbich <deb2001-github@yahoo.de> Reviewed-By:
-
- Apr 18, 2020
-
-
Rich Trott authored
PR-URL: https://github.com/nodejs/node/pull/32832 Reviewed-By:
Myles Borins <myles.borins@gmail.com> Reviewed-By:
-
Nimit authored
Convert callback functions that are anonymous to arrow functions for better readability. Also adjusted the `this` object in places where that was required. PR-URL: https://github.com/nodejs/node/pull/32875 Reviewed-By:
Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By:
-
Nimit authored
elevate v8 namespaces. Leverage `using` semantics for repeated usage of v8 artifacts. PR-URL: https://github.com/nodejs/node/pull/32872 Reviewed-By:
-
Anna Henningsen authored
This fixes the following crash on Windows for me. I don't know why this I only started to see this now, but anyway, the new timeout value is still longer than a week and a half. File "tools/test.py", line 1725, in <module> sys.exit(Main()) File "tools/test.py", line 1701, in Main if RunTestCases(cases_to_run, options.progress, \ options.j, options.flaky_tests): File "tools/test.py", line 923, in RunTestCases return progress.Run(tasks) File "tools/test.py", line 145, in Run thread.join(timeout=10000000) File "C:\Users\anna\AppData\Local\Programs\Python\Python38-32\ \ lib\threading.py", line 1015, in join self._wait_for_tstate_lock(timeout=max(timeout, 0)) File "C:\Users\anna\AppData\Local\Programs\Python\Python38-32\ \ lib\threading.py", line 1027, in _wait_for_tstate_lock elif lock.acquire(block, timeout): OverflowError: timeout value is too large PR-URL: https://github.com/nodejs/node/pull/32868 Reviewed-By:Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By:
Bartosz Sosnowski <bartosz@janeasystems.com>
-
Juan José Arboleda authored
PR-URL: https://github.com/nodejs/node/pull/32854 Reviewed-By:
-
Michael Dawson authored
We missed adding version 6 to the compatibility table when we defined version 6. Add it along with the versions that we know will include version 6. PR-URL: https://github.com/nodejs/node/pull/32829 Reviewed-By:
Chengzhong Wu <legendecas@gmail.com> Reviewed-By:
-
Juan José Arboleda authored
PR-URL: https://github.com/nodejs/node/pull/32906 Reviewed-By:
Rich Trott <rtrott@gmail.com> Reviewed-By:
-
Guy Bedford authored
PR-URL: https://github.com/nodejs/node/pull/31974 Reviewed-By:
Jan Krems <jan.krems@gmail.com> Reviewed-By:
Geoffrey Booth <webmaster@geoffreybooth.com> Reviewed-By:
Michaël Zasso <targos@protonmail.com>
-
William Bonawentura authored
PR-URL: https://github.com/nodejs/node/pull/32657 Reviewed-By:
-
- Apr 17, 2020
-
-
Qinhui Chen authored
Delete useless module in parent module: parent.children array when error occurs, so that it can be garbage collected. Fixes: https://github.com/nodejs/node/issues/32836 PR-URL: https://github.com/nodejs/node/pull/32837 Reviewed-By:
Andrey Pechkurov <apechkurov@gmail.com>
-
Daniel Bevenius authored
This commit removes unused using declarations in src/node_worker.cc. PR-URL: https://github.com/nodejs/node/pull/32840 Reviewed-By:
-
Michael Dawson authored
- add backticks around use of NULL - convert from null to NULL where we mean NULL Signed-off-by:
Christopher Hiller <boneskull@boneskull.com>
-
Richard Lau authored
When running `make test-v8` V8's `gclient sync` converts folders under `deps/v8/third_party` into git repositories. Unfortunately the files that were checked in under `deps/v8/third_party/zlib` have been modified from the upstream Chromium repository (some files have been deleted and there are whitespace differences in some of the files that were kept) so whenever the Node.js source tree is hard reset/checked out `gclient sync` notices there are unstaged changes as the files in the Node.js source tree do not match those of the upstream Chromium third party zlib commit. Signed-off-by:
Matheus Marchini <mat@mmarchini.me> Reviewed-By:
-
Gabriel Schulhof authored
This reverts commit aeb7084f. The solution creates incorrect behaviour on Windows. Re: https://github.com/nodejs/node-addon-api/pull/697#issuecomment-612993476 Signed-off-by:
-
- Apr 16, 2020
-
-
daemon1024 authored
PR-URL: https://github.com/nodejs/node/pull/32692 Fixes: https://github.com/nodejs/node/issues/32678 Reviewed-By:
-
Robert Nagy authored
Cleans up comments and TODOs and tries to provide a more detail description in regards to error behavior of end(). PR-URL: https://github.com/nodejs/node/pull/32839 Reviewed-By:
-
Robert Nagy authored
destroy w/ callback was previously used by node core. This is no longer the case and the comments should reflect this to avoid confusion. PR-URL: https://github.com/nodejs/node/pull/32808 Reviewed-By:
-
Xu Meng authored
To skip some tests on IBMi PASE, we use uname to detect the true os name, but on Windows machines there is no uname available. PR-URL: https://github.com/nodejs/node/pull/32833 Reviewed-By:
Tobias Nießen <tniessen@tnie.de> Reviewed-By:
Sam Roberts <vieuxtech@gmail.com> Reviewed-By:
Ujjwal Sharma <ryzokuken@disroot.org> Reviewed-By:
-
Robert Nagy authored
Significantly simplified Transform stream implementation by using mostly standard stream code. PR-URL: https://github.com/nodejs/node/pull/32763 Reviewed-By:
-
Richard Lau authored
Update cross compiler machine for Linux armv7 to Ubuntu 18.04. Signed-off-by:
Beth Griggs <Bethany.Griggs@uk.ibm.com> Reviewed-By:
-
Richard Lau authored
Releases built on Centos/RHEL have been updated to use devtoolset-8. Signed-off-by:
-
Richard Lau authored
Based on feedback from Joyent, the SmartOS builds are largely unusable without bundling all of the libraries used to build the binary so are being dropped starting from Node.js 14. We will still test SmartOS in the CI. Signed-off-by:
-
- Apr 15, 2020
-
-
Anna Henningsen authored
Refs: https://github.com/nodejs/node/pull/32745#discussion_r406564208 PR-URL: https://github.com/nodejs/node/pull/32815 Reviewed-By:
-
Robert Nagy authored
Refer back to streams docs for further and more accurate description of behavior details. Refs: https://github.com/nodejs/node/issues/31916 PR-URL: https://github.com/nodejs/node/pull/32811 Reviewed-By:
Denys Otrishko <shishugi@gmail.com>
-
Robert Nagy authored
destroy w/ is undocumented API which also will cause a race if the stream is already destroying and potentially invoking the callback too early and withou error. PR-URL: https://github.com/nodejs/node/pull/32809 Reviewed-By:
-
Sam Roberts authored
Refs: https://github.com/nodejs/node/issues/27611#issuecomment-613100468 PR-URL: https://github.com/nodejs/node/pull/32828 Reviewed-By:
-
Robert Nagy authored
Remove nesting in favor of early returns. PR-URL: https://github.com/nodejs/node/pull/32801 Reviewed-By:
-
Anna Henningsen authored
Refs: https://github.com/nodejs/node/pull/32797#discussion_r407222290 PR-URL: https://github.com/nodejs/node/pull/32798 Reviewed-By:
-
Anna Henningsen authored
Suggest using `--trace-warnings` or `--trace-deprecation` the first time a warning is emitted without a stack trace, similar to how we suggest `--trace-uncaught` when printing uncaught exceptions without a stack trace. PR-URL: https://github.com/nodejs/node/pull/32797 Reviewed-By:
-
Jesus Hernandez authored
PR-URL: https://github.com/nodejs/node/pull/32727 Reviewed-By:
-
Robert Nagy authored
Ensure that the socket is not destroyed when the 'end' event is emitted. Refs: https://github.com/nodejs/node/pull/32780#issuecomment-612602551 PR-URL: https://github.com/nodejs/node/pull/32794 Reviewed-By:
-
