Skip to content

Conversation

@ofrobots
Copy link
Contributor

This is for next+1. This picks up the fix for the out-of-band write unicode-decoder issue.

R=@bnoordhuis?

unicode-decoder: fix out-of-band write in utf16 (v8 issue 4274).

Include Harmony Array/TypedArray methods unconditionally (Chromium
issue 504629).
@bnoordhuis
Copy link
Member

Rubber-stamp LGTM. Can you run the CI before merging?

@ofrobots
Copy link
Contributor Author

@mscdex mscdex added the v8 engine Issues and PRs related to the V8 dependency. label Jul 15, 2015
@ofrobots
Copy link
Contributor Author

Second CI https://jenkins-iojs.nodesource.com/job/iojs+any-pr+multi/154/ because the first one hit machine issues. TLS failures on win2008 look unrelated and are failing on other CIs too.

I will land this tomorrow.

ofrobots added a commit that referenced this pull request Jul 16, 2015
unicode-decoder: fix out-of-band write in utf16 (v8 issue 4274).

Include Harmony Array/TypedArray methods unconditionally (Chromium
issue 504629).

PR-URL: #2183
Reviewed-By: Ben Noordhuis <[email protected]>
@Fishrock123
Copy link
Contributor

@ofrobots did you land this?

@ofrobots ofrobots closed this Jul 16, 2015
@ofrobots ofrobots deleted the next+1 branch July 16, 2015 19:41
@ofrobots
Copy link
Contributor Author

Yes, landed in a6b6295.

@ChALkeR
Copy link
Member

ChALkeR commented Aug 15, 2015

Just in case: this technically is a SEMVER-MAJOR due to backwards incompatible changes in v8 (finally enforcing TDZ in for loops).

See #2203.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

v8 engine Issues and PRs related to the V8 dependency.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants