v13.3.0 proposal

.eslintrc.js

@@ -118,12 +118,14 @@ module.exports = {
     'no-class-assign': 'error',
     'no-confusing-arrow': 'error',
     'no-const-assign': 'error',
+    'no-constructor-return': 'error',
     'no-control-regex': 'error',
     'no-debugger': 'error',
     'no-delete-var': 'error',
     'no-dupe-args': 'error',
     'no-dupe-class-members': 'error',
     'no-dupe-keys': 'error',
+    'no-dupe-else-if': 'error',
     'no-duplicate-case': 'error',
     'no-duplicate-imports': 'error',
     'no-empty-character-class': 'error',
@@ -247,6 +249,7 @@ module.exports = {
     'no-return-await': 'error',
     'no-self-assign': 'error',
     'no-self-compare': 'error',
+    'no-setter-return': 'error',
     'no-shadow-restricted-names': 'error',
     'no-tabs': 'error',
     'no-template-curly-in-string': 'error',
@@ -279,6 +282,10 @@ module.exports = {
     'one-var': ['error', { initialized: 'never' }],
     'one-var-declaration-per-line': 'error',
     'operator-linebreak': ['error', 'after'],
+    'padding-line-between-statements': [
+      'error',
+      { blankLine: 'always', prev: 'function', next: 'function' },
+    ],
     'prefer-const': ['error', { ignoreReadBeforeAssign: true }],
     'quotes': ['error', 'single', { avoidEscape: true }],
     'quote-props': ['error', 'consistent'],

AUTHORS

@@ -2866,5 +2866,84 @@ akitsu-sanae <akitsu.sanae@gmail.com>
 Minuk Park <parkm86@gmail.com>
 Jim Schlight <jim@inspiredware.com>
 Theotime Poisseau <theotime.poisseau@gmail.com>
+Alex Zherdev <alex_zherdev@acl.com>
+dev-313 <shekhawat.dev313@gmail.com>
+Michael Perrotte <mike@npmjs.com>
+Alexandre Ferrando <alferpal@gmail.com>
+Loris Zinsou <lzinsou@protonmail.com>
+Jizu Sun <jizu.sun@sap.com>
+AshCripps <Ashley.Cripps@ibm.com>
+garygsc <garygsc@gmail.com>
+Patrick Housley <patrick.f.housley@gmail.com>
+Artem Maksimov <temamaksimov@gmail.com>
+Nolik <nolikfriendly@gmail.com>
+palmires <palmires@example.com>
+Vadim Gorbachev <bmsdave@gmail.com>
+galina.prokofeva <g.k.prokofeva@gmail.com>
+Nadya <nadin.ts@gmail.com>
+PerfectPan <perfectpan325@gmail.com>
+peze <cpj1106@gmail.com>
+Vladislav Botvin <darkvlados@gmail.com>
+Ilia Safronov <ilia.a.safronov@gmail.com>
+Dmitriy Kikinskiy <di9@ytka.org>
+telenord <nordtelecom.yandex.ru>
+alexahdp <alexahdp@gmail.com>
+Daniil Pletnev <divided.joy@gmail.com>
+Grigoriy Levanov <glevanov@gmail.com>
+Simon Schick <demwizzy@gmail.com>
+Albert Wang <git@albertyw.com>
+Kenza Houmani <khoumani@bloomberg.net>
+mkdorff <mkdorff@outlook.com>
+xefimx <xefimix@gmail.com>
+garygsc <GaryGSC@users.noreply.github.com>
+Susana Ferreira <ssncferreira@gmail.com>
+Xavier Redondo <xavier.redondo@axa.com>
+Duncan Healy <duncan.healy@gmail.com>
+SoulMonk <fsoulmonk@gmail.com>
+Kerry Mahne <kerry.mahne@gmail.com>
+nathias <n@th1.as>
+Nikolay Krashnikov <1090219@mail.ru>
+daern91 <daniel_eriksson_8@hotmail.com>
+JL Phillips <jlphillips@rms.com>
+Jure Stepisnik <jure.stepisnik@tretton37.com>
+Raoul Jaeckel <raoul.jaeckel@daimler.com>
+Kyriakos Markakis <kyriakos@marqui.dev>
+Guilherme Goncalves <gsg@ggoncalves.me>
+Jesse O'Connor <jessekoconnor@gmail.com>
+VinceOPS <vince.g.135@gmail.com>
+David OLIVIER <davidolivier1337@gmail.com>
+Maria Stogova <masha.stogova@gmail.com>
+Taylor Gagne <tgagne@netsurion.com>
+Kirlat <kirlat@yula.media>
+Lucas Recknagel <lucas91@me.com>
+Oliver Belaifa <oliver@belaifa.de>
+guzhizhou <guzhizhou28@gmail.com>
+Paolo Ceschi Berrini <pc3b3r@gmail.com>
+Jing Lin <selinajinglin09@gmail.com>
+poutch <damien.pucheu@hotmail.fr>
+Tembrechts <Tim.embrechts@emby.ey>
+Jon Church <me@jonchurch.com>
+Àlvar Pérez <alvarperez92@gmail.com>
+Daniel Schuech <dschuech@gmail.com>
+Dimitris Ktistakis <dimitris@workable.com>
+Dries Stelten <dries.stelten@telenet.be>
+Vladimir Adamic <vladimir.adamic@gmail.com>
+Shubham Chaturvedi <19shubham11@gmail.com>
+Jamar Torres <Jetorres88@gmail.com>
+Luis Camargo <luiska1993@gmail.com>
+Herrmann, Rene R. (656) <rene.r.herrmann@daimler.com>
+Semir Ajruli <ajruli.semir@gmail.com>
+matijagaspar <matija.gaspar@gmail.com>
+Aldo Ambrosioni <aldo.ambrosioni@accenture.com>
+Nazar Malyy <nazar.malyy@gmail.com>
+Dennis Saenger <node@mail.ds82.de>
+ryan jarvinen <ryan.jarvinen@gmail.com>
+Maria Emmanouil <emmanouil.maria@gmail.com>
+Tijl Claessens <tijl.claessens@hotmail.com>
+EmaSuriano <emanuel.suriano@gmail.com>
+Chris Oyler <christopher.oyler@gmail.com>
+Jesper Ek <deadbeef84@gmail.com>
+Luciano <lucianomammino@gmail.com>
+jens-cappelle <jens.jan.cappelle@gmail.com>
 
 # Generated by tools/update-authors.js

BUILDING.md

@@ -116,7 +116,7 @@ platforms. This is true regardless of entries in the table below.
 | macOS            | x64              | >= 10.11                        | Tier 1       |                                   |
 | SmartOS          | x64              | >= 18                           | Tier 2       |                                   |
 | AIX              | ppc64be >=power7 | >= 7.2 TL02                     | Tier 2       |                                   |
-| FreeBSD          | x64              | >= 11                           | Experimental | Downgraded as of Node.js 12       |
+| FreeBSD          | x64              | >= 11                           | Experimental | Downgraded as of Node.js 12  <sup>[7](#fn7)</sup>     |
 
 <em id="fn1">1</em>: GCC 6 is not provided on the base platform, users will
   need the
@@ -150,6 +150,10 @@ are provided. However, tests in our infrastructure only run on WoW64.
 Furthermore, compiling on x86 Windows is currently considered Experimental and
 may not be possible.
 
+<em id="fn7">7</em>: The default FreeBSD 12.0 compiler is Clang 6.0.1, but
+FreeBSD 12.1 upgrades to 8.0.1. Other Clang/LLVM versions are provided
+via the system's package manager, including Clang 9.0.
+
 ### Supported toolchains
 
 Depending on the host platform, the selection of toolchains may vary.
@@ -214,9 +218,10 @@ Supported platforms and toolchains change with each major version of Node.js.
 This document is only valid for the current major version of Node.js.
 Consult previous versions of this document for older versions of Node.js:
 
+* [Node.js 13](https://github.com/nodejs/node/blob/v13.x/BUILDING.md)
+* [Node.js 12](https://github.com/nodejs/node/blob/v12.x/BUILDING.md)
 * [Node.js 10](https://github.com/nodejs/node/blob/v10.x/BUILDING.md)
 * [Node.js 8](https://github.com/nodejs/node/blob/v8.x/BUILDING.md)
-* [Node.js 6](https://github.com/nodejs/node/blob/v6.x/BUILDING.md)
 
 ## Building Node.js on supported platforms
 

CHANGELOG.md

@@ -30,7 +30,8 @@ release.
 </tr>
 <tr>
     <td valign="top">
-<b><a href="doc/changelogs/CHANGELOG_V13.md#13.2.0">13.2.0</a></b><br/>
+<b><a href="doc/changelogs/CHANGELOG_V13.md#13.3.0">13.3.0</a></b><br/>
+<a href="doc/changelogs/CHANGELOG_V13.md#13.2.0">13.2.0</a><br/>
 <a href="doc/changelogs/CHANGELOG_V13.md#13.1.0">13.1.0</a><br/>
 <a href="doc/changelogs/CHANGELOG_V13.md#13.0.1">13.0.1</a><br/>
 <a href="doc/changelogs/CHANGELOG_V13.md#13.0.0">13.0.0</a><br/>

COLLABORATOR_GUIDE.md

@@ -187,7 +187,9 @@ Do not land any pull requests without passing (green or yellow) CI runs. If
 there are CI failures unrelated to the change in the pull request, try "Resume
 Build". It is in the left navigation of the relevant `node-test-pull-request`
 job. It will preserve all the green results from the current job but re-run
-everything else.
+everything else. Start a fresh CI if more than seven days have elapsed since
+the original failing CI as the compiled binaries for the Windows and ARM
+platforms are only kept for seven days.
 
 Some of the CI Jobs may require `GIT_REMOTE_REF` which is the remote portion
 of Git refspec. To specify the branch this way `refs/heads/BRANCH` is used

CPP_STYLE_GUIDE.md

@@ -1,5 +1,8 @@
 # C++ Style Guide
 
+See also the [C++ codebase README](src/README.md) for C++ idioms in the Node.js
+codebase not related to stylistic issues.
+
 ## Table of Contents
 
 * [Guides and References](#guides-and-references)

LICENSE

@@ -1503,3 +1503,28 @@ The externally maintained libraries used by Node.js are:
     ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR
     IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
   """
+
+- uvwasi, located at deps/uvwasi, is licensed as follows:
+  """
+    MIT License
+
+    Copyright (c) 2019 Colin Ihrig and Contributors
+
+    Permission is hereby granted, free of charge, to any person obtaining a copy
+    of this software and associated documentation files (the "Software"), to deal
+    in the Software without restriction, including without limitation the rights
+    to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+    copies of the Software, and to permit persons to whom the Software is
+    furnished to do so, subject to the following conditions:
+
+    The above copyright notice and this permission notice shall be included in all
+    copies or substantial portions of the Software.
+
+    THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+    IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+    FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+    AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+    LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+    OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+    SOFTWARE.
+  """

Makefile

@@ -489,6 +489,7 @@ test-all-valgrind: test-build
 test-all-suites: | clear-stalled test-build bench-addons-build doc-only ## Run all test suites.
 	$(PYTHON) tools/test.py $(PARALLEL_ARGS) --mode=$(BUILDTYPE_LOWER) test/*
 
+# CI_* variables should be kept synchronized with the ones in vcbuild.bat
 CI_NATIVE_SUITES ?= addons js-native-api node-api
 CI_JS_SUITES ?= default
 ifeq ($(node_use_openssl), false)
@@ -745,7 +746,7 @@ $(LINK_DATA): $(wildcard lib/*.js) tools/doc/apilinks.js
 	$(call available-node, $(gen-apilink))
 
 out/doc/api/%.json out/doc/api/%.html: doc/api/%.md tools/doc/generate.js \
-	tools/doc/html.js tools/doc/json.js tools/doc/apilinks.js | $(LINK_DATA)
+	tools/doc/markdown.js tools/doc/html.js tools/doc/json.js tools/doc/apilinks.js | $(LINK_DATA)
 	$(call available-node, $(gen-api))
 
 out/doc/api/all.html: $(apidocs_html) tools/doc/allhtml.js \

README.md

@@ -49,7 +49,7 @@ Looking for help? Check out the
   each October.
 * **LTS**: Releases that receive Long-term Support, with a focus on stability
   and security. Every even-numbered major version will become an LTS release.
-  LTS releases receive 18 months of _Active LTS_ support and a further 12 months
+  LTS releases receive 12 months of _Active LTS_ support and a further 18 months
   of _Maintenance_. LTS release lines have alphabetically-ordered codenames,
   beginning with v4 Argon. There are no breaking changes or feature additions,
   except in some special circumstances.

benchmark/process/next-tick-breadth-args.js

@@ -13,16 +13,19 @@ function main({ n }) {
     if (j === n)
       bench.end(n);
   }
+
   function cb2(arg1, arg2) {
     j++;
     if (j === n)
       bench.end(n);
   }
+
   function cb3(arg1, arg2, arg3) {
     j++;
     if (j === n)
       bench.end(n);
   }
+
   function cb4(arg1, arg2, arg3, arg4) {
     j++;
     if (j === n)

benchmark/process/next-tick-depth-args.js

@@ -20,6 +20,7 @@ function main({ n }) {
     } else
       bench.end(n);
   }
+
   function cb3(arg1, arg2, arg3) {
     if (--counter) {
       if (counter % 4 === 0)
@@ -33,6 +34,7 @@ function main({ n }) {
     } else
       bench.end(n);
   }
+
   function cb2(arg1, arg2) {
     if (--counter) {
       if (counter % 4 === 0)
@@ -46,6 +48,7 @@ function main({ n }) {
     } else
       bench.end(n);
   }
+
   function cb1(arg1) {
     if (--counter) {
       if (counter % 4 === 0)

benchmark/streams/writable-manywrites.js

@@ -4,14 +4,19 @@ const common = require('../common');
 const Writable = require('stream').Writable;
 
 const bench = common.createBenchmark(main, {
-  n: [2e6]
+  n: [2e6],
+  sync: ['yes', 'no']
 });
 
-function main({ n }) {
+function main({ n, sync }) {
   const b = Buffer.allocUnsafe(1024);
   const s = new Writable();
+  sync = sync === 'yes';
   s._write = function(chunk, encoding, cb) {
-    cb();
+    if (sync)
+      cb();
+    else
+      process.nextTick(cb);
   };
 
   bench.start();

benchmark/timers/set-immediate-breadth-args.js

@@ -12,7 +12,9 @@ function main({ n }) {
   });
 
   function cb1(arg1) {}
+
   function cb2(arg1, arg2) {}
+
   function cb3(arg1, arg2, arg3) {}
 
   bench.start();

benchmark/timers/set-immediate-depth-args.js

@@ -21,6 +21,7 @@ function main({ n }) {
         setImmediate(cb1, n);
     }
   }
+
   function cb2(n, arg2) {
     if (--n) {
       if (n % 3 === 0)
@@ -31,6 +32,7 @@ function main({ n }) {
         setImmediate(cb1, n);
     }
   }
+
   function cb1(n) {
     if (--n) {
       if (n % 3 === 0)

benchmark/timers/timers-breadth-args.js

@@ -12,16 +12,19 @@ function main({ n }) {
     if (j === n)
       bench.end(n);
   }
+
   function cb2(arg1, arg2) {
     j++;
     if (j === n)
       bench.end(n);
   }
+
   function cb3(arg1, arg2, arg3) {
     j++;
     if (j === n)
       bench.end(n);
   }
+
   function cb4(arg1, arg2, arg3, arg4) {
     j++;
     if (j === n)

benchmark/timers/timers-timeout-nexttick.js

@@ -19,9 +19,11 @@ function main({ n }) {
   function cb() {
     process.nextTick(counter);
   }
+
   function cb2() {
     process.nextTick(counter);
   }
+
   function counter() {
     count++;
     if (count === n)

benchmark/timers/timers-timeout-pooled.js

@@ -20,6 +20,7 @@ function main({ n }) {
     if (count === n)
       bench.end(n);
   }
+
   function cb2() {
     count++;
     if (count === n)

benchmark/timers/timers-timeout-unpooled.js

@@ -20,6 +20,7 @@ function main({ n }) {
     if (count === n)
       bench.end(n);
   }
+
   function cb2() {
     count++;
     if (count === n)

common.gypi

@@ -39,7 +39,7 @@
 
     # Reset this number to 0 on major V8 upgrades.
     # Increment by one for each non-official patch applied to deps/v8.
-    'v8_embedder_string': '-node.20',
+    'v8_embedder_string': '-node.23',
 
     ##### V8 defaults for Node.js #####
 

configure

@@ -20,7 +20,7 @@ del _
 import sys
 from distutils.spawn import find_executable
 
-print('Node configure: Found Python {0}.{1}.{2}...'.format(*sys.version_info))
+print('Node.js configure: Found Python {0}.{1}.{2}...'.format(*sys.version_info))
 acceptable_pythons = ((3, 8), (3, 7), (3, 6), (3, 5), (2, 7))
 if sys.version_info[:2] in acceptable_pythons:
   import configure