Commits · 08809f28ad797b914bad2dcd21eefba898eaaa4d · github.com / v8-riscv / node

May 02, 2017

James M Snell authored Apr 25, 2017

PR-URL: https://github.com/nodejs/node/pull/12644


Reviewed-By: Brian White <mscdex@mscdex.net>

08809f28

streams: refactor BufferList into ES6 class · e2199e0f

James M Snell authored Apr 24, 2017

PR-URL: https://github.com/nodejs/node/pull/12644


Reviewed-By: Brian White <mscdex@mscdex.net>

e2199e0f

module: refactor internal/module export style · ed0716f0

James M Snell authored Apr 24, 2017

PR-URL: https://github.com/nodejs/node/pull/12644


Reviewed-By: Brian White <mscdex@mscdex.net>

ed0716f0

process: refactor internal/process.js export style · 579ff2a4

James M Snell authored Apr 24, 2017

PR-URL: https://github.com/nodejs/node/pull/12644


Reviewed-By: Brian White <mscdex@mscdex.net>

579ff2a4

May 01, 2017

src: support domains with empty labels · 0f58d3cb

Daijiro Wachi authored May 01, 2017

Follow the spec of domainToASCII/domainToUnicode in whatwg, and
synchronise WPT url test data.

Refs: https://github.com/w3c/web-platform-tests/pull/5397
PR-URL: https://github.com/nodejs/node/pull/12707


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>

0f58d3cb

deps: cherry-pick bfae9db from upstream v8 · 6ed791c6

Ben Noordhuis authored Apr 28, 2017

Original commit message:

    Update postmortem metadata generator.

    Add PropertyDetails::AttributesField +
    PropertyDetails::LocationField.

    Review-Url: https://codereview.chromium.org/2842843004
    Cr-Commit-Position: refs/heads/master@{#44889}

PR-URL: https://github.com/nodejs/node/pull/12722
Refs: https://github.com/nodejs/llnode/issues/81


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

6ed791c6

test: move test to sequential for reliability · 0833d31e

Rich Trott authored Apr 27, 2017

test-https-set-timeout-server fails under load. Move it to sequential so
it is not competing with other tests.

PR-URL: https://github.com/nodejs/node/pull/12704
Fixes: https://github.com/nodejs/node/issues/10130


Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>

0833d31e

Apr 30, 2017

doc: fix formatting of TOC · aa3eab00

Refael Ackermann authored Apr 28, 2017

PR-URL: https://github.com/nodejs/node/pull/12731


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>

aa3eab00

test: fix permanent deoptimizations · b6da2257

Brian White authored Apr 24, 2017

PR-URL: https://github.com/nodejs/node/pull/12456


Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

b6da2257

repl: fix permanent deoptimizations · b45abfda

Brian White authored Apr 24, 2017

PR-URL: https://github.com/nodejs/node/pull/12456


Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

b45abfda

readline: fix permanent deoptimizations · 4484e8f7

Brian White authored Apr 24, 2017

PR-URL: https://github.com/nodejs/node/pull/12456


Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

4484e8f7

net: fix permanent deoptimizations · 3c098ee7

Brian White authored Apr 24, 2017

PR-URL: https://github.com/nodejs/node/pull/12456


Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

3c098ee7

util: fix permanent deoptimizations · d5925af8

Brian White authored Apr 24, 2017

PR-URL: https://github.com/nodejs/node/pull/12456


Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

d5925af8

url: fix permanent deoptimizations · 8491c705

Brian White authored Apr 24, 2017

PR-URL: https://github.com/nodejs/node/pull/12456


Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

8491c705

process: fix permanent deoptimizations · a641b7cf

Brian White authored Apr 24, 2017

PR-URL: https://github.com/nodejs/node/pull/12456


Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

a641b7cf

cluster: fix permanent deoptimizations · 033ea997

Brian White authored Apr 24, 2017

PR-URL: https://github.com/nodejs/node/pull/12456


Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

033ea997

fs: fix permanent deoptimizations · e8a42907

Brian White authored Apr 24, 2017

PR-URL: https://github.com/nodejs/node/pull/12456


Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

e8a42907

stream: fix permanent deoptimizations · e9c02c67

Brian White authored Apr 24, 2017

PR-URL: https://github.com/nodejs/node/pull/12456


Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

e9c02c67

http: fix permanent deoptimizations · e2833199

Brian White authored Apr 24, 2017

PR-URL: https://github.com/nodejs/node/pull/12456


Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

e2833199

build: add target for checking for perm deopts · 7a5bac5d

Brian White authored Apr 17, 2017

PR-URL: https://github.com/nodejs/node/pull/12456


Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

7a5bac5d

test: fix test filenames · 91ccea65

Brian White authored Apr 17, 2017

PR-URL: https://github.com/nodejs/node/pull/12456


Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

91ccea65

test: replace indexOf with includes · 01422769

gwer authored Apr 23, 2017

Start the transition to Array.prototype.includes() and
String.prototype.includes().  This commit refactors most of the
comparisons of Array.prototype.indexOf() and String.prototype.indexOf()
return values with -1 to the former methods in tests.

PR-URL: https://github.com/nodejs/node/pull/12604
Refs: https://github.com/nodejs/node/issues/12586


Reviewed-By: Alexey Orlenko <eaglexrlnk@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>

01422769

test: add inspect-brk option to cluster module · 0324ac68

dave-k authored Apr 18, 2017

Ensure that cluster interoperates with the --inspect-brk option.
This does not test for --debug-brk.

Fixes: https://github.com/nodejs/node/issues/11420
PR-URL: https://github.com/nodejs/node/pull/12503


Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

0324ac68

process: cast promise rejection reason to string · a3132b0a

Cameron Little authored Mar 13, 2017

The unhandled promise rejection warning uses a template literal and
prints the reason a promise was rejected. If rejecting with a symbol,
the symbol failed to convert to a string and the process crashed. Now,
symbols are casted to strings and the process does not crash.

Fixes: https://github.com/nodejs/node/issues/11637
PR-URL: https://github.com/nodejs/node/pull/11640


Reviewed-By: Anna Henningsen <anna@addaleax.net>

a3132b0a

test: support multiple warnings in checkWarning · 74f61e8e

Cameron Little authored Mar 13, 2017

This allows the common.checkWarning() test method to accept a map of
warning names to description(s), to allow testing code that generates
multiple types of warnings.

PR-URL: https://github.com/nodejs/node/pull/11640


Reviewed-By: Anna Henningsen <anna@addaleax.net>

74f61e8e

doc: fixup the collaborators list · 765be6c2

Alexey Orlenko authored Apr 30, 2017

* Add the personal pronoun for @aqrln.
* Fix incorrectly ordered @lucamaraschi entry.

PR-URL: https://github.com/nodejs/node/pull/12750


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>

765be6c2

v8: fix gcc 7 build errors · b07e1a82

Zuzana Svetlikova authored Apr 26, 2017

Porting https://github.com/nodejs/node/pull/12392 to master

Ref: https://github.com/nodejs/node/pull/12392
Fixes: https://github.com/nodejs/node/issues/10388
PR-URL: https://github.com/nodejs/node/pull/12676


Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Anna Henningsen <anna@addaleax.net>

b07e1a82

url: refactor binding imports in internal/url · 97ec72b7

James M Snell authored Apr 27, 2017

PR-URL: https://github.com/nodejs/node/pull/12717


Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>

97ec72b7

url: move to module.exports = {} pattern · b331ba6c

James M Snell authored Apr 27, 2017

PR-URL: https://github.com/nodejs/node/pull/12717


Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>

b331ba6c

Apr 29, 2017

test: improve test-tcp-wrap-listen · 7c238be1

alohaglenn authored Apr 22, 2017

PR-URL: https://github.com/nodejs/node/pull/12599


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>

7c238be1

test,doc: document `crashOnUnhandledRejection()` · 3d87eca1

Anna Henningsen authored Apr 27, 2017

Add documentation for `common.crashOnUnhandledRejection()`.

Ref: https://github.com/nodejs/node/pull/12489/files/a9c2078a60bc3012dc6156df19772697a56a2517#r113737423
PR-URL: https://github.com/nodejs/node/pull/12699


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>

3d87eca1

src: remove debugger dead code · 719247ff

Michaël Zasso authored Apr 24, 2017

PR-URL: https://github.com/nodejs/node/pull/12621


Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>

719247ff

test: cleanup handles in test_environment · d5db4d25

Anna Henningsen authored Apr 27, 2017

The test fixtures create multiple node::Environments that all use the
uv_default_loop(), and since the test does not clean up the handles
created by Environment::Start(), the default libuv loop structure
contains dangling pointers after the first Environment is freed,
which then means that creating new handles leads to memory corruption.

PR-URL: https://github.com/nodejs/node/pull/12621


Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

d5db4d25

test: add hasCrypto check to test-cli-node-options · 427cd293

Daniel Bevenius authored Apr 27, 2017

Currently when configure --without-ssl the test will throw the following
error:
bad option: --use-openssl-ca

This commit checks if crypto was enabled and skips the crypto related
tests if that is the case.

PR-URL: https://github.com/nodejs/node/pull/12692


Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com&gt;/>

427cd293

benchmark: terminate child process on Windows · a1a54ca3

Rich Trott authored Apr 25, 2017

test-benchmark-child-process failures reveal that
child-process-exec-stdout benchmark sometimes leaves around a stray
yes.exe process. Add code to terminate the process.

PR-URL: https://github.com/nodejs/node/pull/12658
Ref: https://github.com/nodejs/node/issues/12560


Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Bartosz Sosnowski <bartosz@janeasystems.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

a1a54ca3

doc: update example in repl.md · d8f8096e

Vse Mozhet Byt authored Apr 27, 2017

Delete unused method call.

PR-URL: https://github.com/nodejs/node/pull/12685
Refs: https://github.com/nodejs/node/pull/6171


Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

d8f8096e

doc: fix examples in repl.md · 8092fb0a

Vse Mozhet Byt authored Apr 27, 2017

* Update an example according to an actual REPL session.
* Replace an arrow function with a common function to hold `this`.

PR-URL: https://github.com/nodejs/node/pull/12684


Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

8092fb0a

test: add relative path to accommodate limit · 0101a8f1

coreybeaumont authored Apr 22, 2017

Found that libuv had a path character limit of 108.

Used path.relative() to set prefix with relative path.

Also added comments explaining the use of a relative path.

PR-URL: https://github.com/nodejs/node/pull/12601


Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>

0101a8f1

v8: refactor struture of v8 module · 1052383f

James M Snell authored Apr 26, 2017

Use the more efficient module.exports = {} pattern,
restructure imports from bindings, requires.

PR-URL: https://github.com/nodejs/node/pull/12681


Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>

1052383f

benchmark: add benchmark for v8.getHeap*Statistics · bed4612c

James M Snell authored Apr 26, 2017

PR-URL: https://github.com/nodejs/node/pull/12681


Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>

bed4612c