Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

v7.10.0 proposal #12775

Merged
merged 182 commits into from
May 3, 2017
Merged

v7.10.0 proposal #12775

merged 182 commits into from
May 3, 2017

Conversation

evanlucas
Copy link
Contributor

@evanlucas evanlucas commented May 1, 2017 •
edited
Loading

Still need to pull in some more commits and will update notable changes shortly

2017-05-02, Version 7.10.0 (Current), @evanlucas

Notable Changes

  • crypto: add randomFill and randomFillSync (Evan Lucas) #10209
  • meta: Added new collaborators
    • add lucamaraschi to collaborators (Luca Maraschi) #12538
    • add DavidCai1993 to collaborators (David Cai) #12435
    • add jkrems to collaborators (Jan Krems) #12427
    • add AnnaMag to collaborators (AnnaMag) #12414
  • process: fix crash when Promise rejection is a Symbol (Cameron Little) #11640
  • url: make WHATWG URL more spec compliant (Timothy Gu) #12507
  • v8:
    • fix stack overflow in recursive method (Ben Noordhuis) #12460
    • fix build errors with g++ 7 (Ben Noordhuis) #12392

Commits

List of commits

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project. tools Issues and PRs related to the tools directory. v7.x v8 engine Issues and PRs related to the V8 dependency. labels May 1, 2017
@evanlucas evanlucas force-pushed the v7.10.0-proposal branch from 1e9fc4b to d730130 Compare May 1, 2017 16:14
bnoordhuis and others added 22 commits May 1, 2017 11:16
@bnoordhuis @evanlucas
deps: cherry-pick 79aee39 from upstream v8
e8595c5 
This is a chery-pick if you consider reducing the context to -C2
a cherry-pick; WordIsSmi has been renamed to TaggedIsSmi upstream.

Original commit message:

    [builtins] Fix pointer comparison in ToString builtin.

    This fixes the bogus {Word32Equal} comparison in the ToString
    builtin implementing Object.prototype.toString to be a pointer-size
    {WordEqual} comparison instead. Comparing just the lower half-word
    is insufficient on 64-bit architectures.

    [email protected]
    TEST=mjsunit/regress/regress-crbug-664506
    BUG=chromium:664506

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

Fixes: #12411
PR-URL: #12412
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
@bnoordhuis @evanlucas
v8: fix build errors with g++ 7
25b851b 
This is a local patch because upstream fixed it differently by moving
large chunks of code out of objects.h.  We cannot easily back-port
those changes due to their size and invasiveness.

Fixes: #10388
PR-URL: #12392
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@Trott @evanlucas
buffer,util: refactor for performance
0c0241f 
internal/util.js definied toInteger() and toLength() but they were only
used by buffer.js. Inlining these small functions results in a small but
statistically-significant performance gain.

PR-URL: #12153
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@Trott @evanlucas
tools: replace custom ESLint timers rule
5ccafa2 
ESLint 3.19.0 allows the specification of selectors that represent
disallowed syntax. Replace our custom rule for timer arguments with a
pair of `no-restricted-syntax` option objects.

PR-URL: #12162
Reviewed-By: Teddy Katz <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
@TimothyGu @evanlucas
url: extend URLSearchParams constructor
b4052e6 
PR-URL: #12507
Fixes: #10635
Reviewed-By: James M Snell <[email protected]>
@TimothyGu @evanlucas
doc: document URLSearchParams constructor
e48e00b 
PR-URL: #12507
Ref: whatwg/url#175
Reviewed-By: James M Snell <[email protected]>
@TimothyGu @evanlucas
url: enforce valid UTF-8 in WHATWG parser
7c9ca0f 
This commit implements the Web IDL USVString conversion, which mandates
all unpaired Unicode surrogates be turned into U+FFFD REPLACEMENT
CHARACTER. It also disallows Symbols to be used as USVString per spec.

Certain functions call into C++ methods in the binding that use the
Utf8Value class to access string arguments. Utf8Value already does the
normalization using V8's String::Write, so in those cases, instead of
doing the full USVString normalization, only a symbol check is done
(`'' + val`, which uses ES's ToString, versus `String()` which has
special provisions for symbols).

PR-URL: #12507
Reviewed-By: James M Snell <[email protected]>
@TimothyGu @evanlucas
url: prioritize toString when stringifying
d6fe916 
The ES addition operator calls the ToPrimitive() abstract operation
without hint String, leading a subsequent OrdinaryToPrimitive() to call
valueOf() first on an object rather than the desired toString().

Instead, use template literals which directly call ToString() abstract
operation, per Web IDL spec.

PR-URL: #12507
Fixes: b610a4d "url: enforce valid UTF-8 in WHATWG parser"
Refs: b610a4d#commitcomment-21200056
Refs: https://tc39.github.io/ecma262/#sec-addition-operator-plus-runtime-semantics-evaluation
Refs: https://tc39.github.io/ecma262/#sec-template-literals-runtime-semantics-evaluation
Reviewed-By: James M Snell <[email protected]>
@TimothyGu @evanlucas
url: spec-compliant URLSearchParams serializer
b8ff2c9 
PR-URL: #12507
Reviewed-By: James M Snell <[email protected]>
@TimothyGu @evanlucas
src: remove explicit UTF-8 validity check in url
72a3cac 
This step was never part of the URL Standard's host parser algorithm,
and is rendered unnecessary after IDNA errors are no longer ignored.

PR-URL: #12507
Refs: c2a302c "src: do not ignore IDNA conversion error"
Reviewed-By: James M Snell <[email protected]>
@TimothyGu @evanlucas
querystring: move isHexTable to internal
0cc37c7 
PR-URL: #12507
Reviewed-By: James M Snell <[email protected]>
@TimothyGu @evanlucas
url: spec-compliant URLSearchParams parser
01b8839 
The entire `URLSearchParams` class is now fully spec-compliant.

PR-URL: #12507
Fixes: #10821
Reviewed-By: James M Snell <[email protected]>
@TimothyGu @evanlucas
url: use a class for WHATWG url[context]
4b6097d 
The object is used as a structure, not as a map, which `StorageObject`
was designed for.

PR-URL: #12507
Reviewed-By: James M Snell <[email protected]>
@jasnell @evanlucas
url: add ToObject method to native URL class
c7de98a 
Provides a factory method to convert a native URL class
into a JS URL object.

```c++
Environment* env = ...

URL url("http://example.org/a/b/c?query#fragment");

MaybeLocal<Value> val = url.ToObject(env);
```

PR-URL: #12507
Reviewed-By: James M Snell <[email protected]>
@TimothyGu @evanlucas
src: WHATWG URL C++ parser cleanup
078316c 
- Clarify port state
- Remove scheme flag
- Clarify URL_FLAG_TERMINATED

PR-URL: #12507
Reviewed-By: James M Snell <[email protected]>
@watilde @evanlucas
url: change path parsing for non-special URLs
b8d1a45 
This changes to the way path parsing for non-special URLs.
It allows paths to be empty for non-special URLs and also
takes that into account when serializing.

PR-URL: #12507
Fixes: #11962
Refs: whatwg/url#213
Reviewed-By: James M Snell <[email protected]>
@watilde @evanlucas
test: synchronize WPT url test data
1adb08e 
PR-URL: #12507
Refs: web-platform-tests/wpt#4586
Refs: #11887
Reviewed-By: James M Snell <[email protected]>
@TimothyGu @evanlucas
url: error when domainTo*() is called w/o argument
b9b93f2 
PR-URL: #12507
Reviewed-By: James M Snell <[email protected]>
@mscdex @evanlucas
url: avoid instanceof for WHATWG URL
b50a84a 
PR-URL: #12507
Reviewed-By: James M Snell <[email protected]>
@watilde @evanlucas
url: trim leading slashes of file URL paths
5a27f63 
It should trim the slashes after the colon into three for file URL.

PR-URL: #12507
Refs: web-platform-tests/wpt#5195
Fixes: #11188
Reviewed-By: James M Snell <[email protected]>
@watilde @evanlucas
url: remove javascript URL special case
ffb2ef4 
PR-URL: #12507
Fixes: #11485
Reviewed-By: James M Snell <[email protected]>
@watilde @evanlucas
url: disallow invalid IPv4 in IPv6 parser
29531d2 
PR-URL: #12507
Fixes: #10655
Reviewed-By: James M Snell <[email protected]>
vsemozhetbyt and others added 20 commits May 2, 2017 20:39
@vsemozhetbyt @evanlucas
doc: fix examples in repl.md
360efe4 
* Update an example according to an actual REPL session.
* Replace an arrow function with a common function to hold `this`.

PR-URL: #12684
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@Trott @evanlucas
benchmark: terminate child process on Windows
224fd3a 
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: #12658
Ref: #12560
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: Bartosz Sosnowski <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@addaleax @evanlucas
test,doc: document crashOnUnhandledRejection()
90bba9f 
Add documentation for `common.crashOnUnhandledRejection()`.

Ref: https://github.com/nodejs/node/pull/12489/files/a9c2078a60bc3012dc6156df19772697a56a2517#r113737423
PR-URL: #12699
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Santiago Gimeno <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
@QueueHefner @evanlucas
test: improve test-tcp-wrap-listen
819c131 
PR-URL: #12599
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
@aqrln @evanlucas
doc: fixup the collaborators list
f60a2e9 
* Add the personal pronoun for @aqrln.
* Fix incorrectly ordered @lucamaraschi entry.

PR-URL: #12750
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
@apexskier @evanlucas
test: support multiple warnings in checkWarning
d78adcc 
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: #11640
Reviewed-By: Anna Henningsen <[email protected]>
@apexskier @evanlucas
process: cast promise rejection reason to string
cd54208 
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: #11637
PR-URL: #11640
Reviewed-By: Anna Henningsen <[email protected]>
@mscdex @evanlucas
test: fix test filenames
9da719a 
PR-URL: #12456
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@mscdex @evanlucas
build: add target for checking for perm deopts
caf6506 
PR-URL: #12456
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@mscdex @evanlucas
stream: fix permanent deoptimizations
7d0adc6 
PR-URL: #12456
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@mscdex @evanlucas
fs: fix permanent deoptimizations
84484b7 
PR-URL: #12456
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@mscdex @evanlucas
cluster: fix permanent deoptimizations
933b6b5 
PR-URL: #12456
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@mscdex @evanlucas
process: fix permanent deoptimizations
b573f77 
PR-URL: #12456
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@mscdex @evanlucas
net: fix permanent deoptimizations
4a08288 
PR-URL: #12456
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@mscdex @evanlucas
readline: fix permanent deoptimizations
1ac331b 
PR-URL: #12456
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@mscdex @evanlucas
test: fix permanent deoptimizations
7af2e79 
PR-URL: #12456
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@refack @evanlucas
doc: fix formatting of TOC
c2c6287 
PR-URL: #12731
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: Daijiro Wachi <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
@Trott @evanlucas
test: move test to sequential for reliability
592db37 
test-https-set-timeout-server fails under load. Move it to sequential so
it is not competing with other tests.

PR-URL: #12704
Fixes: #10130
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Santiago Gimeno <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
@zaide-chris @evanlucas
stream: Fixes missing 'unpipe' event
f5a702e 
Currently when the destination emits an 'error', 'finish' or 'close'
event the pipe calls unpipe to emit 'unpipe' and trigger the clean up
of all it's listeners.
When the source emits an 'end' event without {end: false} it calls
end() on the destination leading it to emit a 'close', this will again
lead to the pipe calling unpipe. However the source emitting an 'end'
event along side {end: false} is the only time the cleanup gets ran
directly without unpipe being called. This fixes that so the 'unpipe'
event does get emitted and cleanup in turn gets ran by that event.

Fixes: #11837
PR-URL: #11876
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
@evanlucas
2017-05-02, Version 7.10.0 (Current)
b5e70ae 
Notable changes:

* **crypto**:
  - add randomFill and randomFillSync (Evan Lucas)
    #10209
* **meta**: Added new collaborators
  - add lucamaraschi to collaborators (Luca Maraschi)
    #12538
  - add DavidCai1993 to collaborators (David Cai)
    #12435
  - add jkrems to collaborators (Jan Krems)
    #12427
  - add AnnaMag to collaborators (AnnaMag)
    #12414
* **process**:
  - fix crash when Promise rejection is a Symbol (Cameron Little)
    #11640
* **url**:
  - make WHATWG URL more spec compliant (Timothy Gu)
    #12507
* **v8**:
  - fix stack overflow in recursive method (Ben Noordhuis)
    #12460
  - fix build errors with g++ 7 (Ben Noordhuis)
    #12392

PR-URL: #12775
@evanlucas evanlucas force-pushed the v7.10.0-proposal branch from 84ed5f8 to b5e70ae Compare May 3, 2017 01:44
@evanlucas evanlucas merged commit 69a8053 into v7.x May 3, 2017
@evanlucas evanlucas deleted the v7.10.0-proposal branch May 3, 2017 12:55
evanlucas added a commit that referenced this pull request May 3, 2017
@evanlucas
2017-05-02, Version 7.10.0 (Current)
871fb5a 
Notable changes:

* **crypto**:
  - add randomFill and randomFillSync (Evan Lucas)
    #10209
* **meta**: Added new collaborators
  - add lucamaraschi to collaborators (Luca Maraschi)
    #12538
  - add DavidCai1993 to collaborators (David Cai)
    #12435
  - add jkrems to collaborators (Jan Krems)
    #12427
  - add AnnaMag to collaborators (AnnaMag)
    #12414
* **process**:
  - fix crash when Promise rejection is a Symbol (Cameron Little)
    #11640
* **url**:
  - make WHATWG URL more spec compliant (Timothy Gu)
    #12507
* **v8**:
  - fix stack overflow in recursive method (Ben Noordhuis)
    #12460
  - fix build errors with g++ 7 (Ben Noordhuis)
    #12392

PR-URL: #12775
evanlucas added a commit to evanlucas/nodejs.org that referenced this pull request May 3, 2017
@evanlucas
blog: add release post for v7.10.0 release
a977061 
Ref: nodejs/node#12775
@evanlucas evanlucas mentioned this pull request May 3, 2017
Merged
evanlucas added a commit to nodejs/nodejs.org that referenced this pull request May 3, 2017
@evanlucas
blog: add release post for v7.10.0 release
490e7e5 
Ref: nodejs/node#12775
imyller added a commit to imyller/meta-nodejs that referenced this pull request May 4, 2017
@imyller
2017-05-02, Version 7.10.0 (Current)
eec531e 
    Notable changes:

    * **crypto**:
      - add randomFill and randomFillSync (Evan Lucas)
        nodejs/node#10209
    * **meta**: Added new collaborators
      - add lucamaraschi to collaborators (Luca Maraschi)
        nodejs/node#12538
      - add DavidCai1993 to collaborators (David Cai)
        nodejs/node#12435
      - add jkrems to collaborators (Jan Krems)
        nodejs/node#12427
      - add AnnaMag to collaborators (AnnaMag)
        nodejs/node#12414
    * **process**:
      - fix crash when Promise rejection is a Symbol (Cameron Little)
        nodejs/node#11640
    * **url**:
      - make WHATWG URL more spec compliant (Timothy Gu)
        nodejs/node#12507
    * **v8**:
      - fix stack overflow in recursive method (Ben Noordhuis)
        nodejs/node#12460
      - fix build errors with g++ 7 (Ben Noordhuis)
        nodejs/node#12392

    PR-URL: nodejs/node#12775

Signed-off-by: Ilkka Myller <[email protected]>
anchnk pushed a commit to anchnk/node that referenced this pull request May 6, 2017
@evanlucas
2017-05-02, Version 7.10.0 (Current)
575955f 
Notable changes:

* **crypto**:
  - add randomFill and randomFillSync (Evan Lucas)
    nodejs#10209
* **meta**: Added new collaborators
  - add lucamaraschi to collaborators (Luca Maraschi)
    nodejs#12538
  - add DavidCai1993 to collaborators (David Cai)
    nodejs#12435
  - add jkrems to collaborators (Jan Krems)
    nodejs#12427
  - add AnnaMag to collaborators (AnnaMag)
    nodejs#12414
* **process**:
  - fix crash when Promise rejection is a Symbol (Cameron Little)
    nodejs#11640
* **url**:
  - make WHATWG URL more spec compliant (Timothy Gu)
    nodejs#12507
* **v8**:
  - fix stack overflow in recursive method (Ben Noordhuis)
    nodejs#12460
  - fix build errors with g++ 7 (Ben Noordhuis)
    nodejs#12392

PR-URL: nodejs#12775
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project. tools Issues and PRs related to the tools directory. v8 engine Issues and PRs related to the V8 dependency.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.