Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1344)

Issue 1801573002: Version 5.0.71.17 (cherry-pick) (Closed)

Created:
4 years, 9 months ago by Yang
Modified:
4 years, 9 months ago
Reviewers:
Michael Hablich
CC:
v8-reviews_googlegroups.com, Paweł Hajdan Jr., ulan
Base URL:
https://chromium.googlesource.com/v8/v8.git@5.0
Target Ref:
refs/pending/branch-heads/5.0
Project:
v8
Visibility:
Public.

Description

Version 5.0.71.17 (cherry-pick) Merged f9d7c711afe0389e9c6c48f6653312017109cb6f Merged 2947b2fa6fcebb349f66818b15b063a32ef71b05 Merged d1f68f776eafa62d4bd1cc3465d18c2e47473381 [regexp] Fix off-by-one in CharacterRange::Negate. Fix compile error on arm. [regexp] fix bogus assertion in CharacterRange constructor. BUG=chromium:592343, chromium:593282 LOG=N TBR=hablich@chromium.org Committed: https://chromium.googlesource.com/v8/v8/+/c8a0cf00a399184ec1b51140a7a241fa4d2ad13a

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+122 lines, -64 lines) Patch
M include/v8-version.h View 1 chunk +1 line, -1 line 0 comments Download
M src/regexp/jsregexp.h View 1 chunk +3 lines, -1 line 0 comments Download
M src/regexp/jsregexp.cc View 12 chunks +29 lines, -26 lines 0 comments Download
M src/regexp/regexp-ast.h View 2 chunks +2 lines, -1 line 0 comments Download
M test/cctest/test-regexp.cc View 8 chunks +37 lines, -35 lines 0 comments Download
A test/mjsunit/regress/regress-crbug-592343.js View 1 chunk +12 lines, -0 lines 0 comments Download
A test/mjsunit/regress/regress-crbug-593282.js View 1 chunk +38 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (2 generated)
Yang
4 years, 9 months ago (2016-03-14 09:34:03 UTC) #1
Yang
4 years, 9 months ago (2016-03-14 10:10:48 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
c8a0cf00a399184ec1b51140a7a241fa4d2ad13a.

Powered by Google App Engine
This is Rietveld 408576698