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

Issue 13016: Allow [a-\d] in RegExp parser. (Closed)

Created:
12 years ago by Lasse Reichstein
Modified:
9 years, 6 months ago
CC:
v8-dev
Visibility:
Public.

Description

Allow the non-spec [a-\d] (i.e., ending a range with a character class) in RegExps.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+34 lines, -37 lines) Patch
M src/parser.cc View 4 chunks +33 lines, -35 lines 0 comments Download
M test/cctest/test-regexp.cc View 2 chunks +1 line, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Lasse Reichstein
Small review, please.
12 years ago (2008-12-01 13:27:26 UTC) #1
Christian Plesner Hansen
12 years ago (2008-12-01 14:20:34 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698