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

Issue 12473: * Complete case independent support in Irregexp. (Closed)

Created:
12 years ago by Erik Corry
Modified:
9 years, 7 months ago
CC:
v8-dev
Visibility:
Public.

Description

* Complete case independent support in Irregexp. Committed: http://code.google.com/p/v8/source/detail?r=853

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+29 lines, -6 lines) Patch
M src/flag-definitions.h View 1 chunk +0 lines, -1 line 0 comments Download
M src/jsregexp.h View 2 chunks +8 lines, -0 lines 1 comment Download
M src/jsregexp.cc View 5 chunks +21 lines, -5 lines 1 comment Download

Messages

Total messages: 2 (0 generated)
Erik Corry
12 years ago (2008-11-26 14:36:17 UTC) #1
Christian Plesner Hansen
12 years ago (2008-11-27 06:49:39 UTC) #2
Lgtm

http://codereview.chromium.org/12473/diff/1/4
File src/jsregexp.cc (right):

http://codereview.chromium.org/12473/diff/1/4#newcode476
Line 476: //PrintF("\n\nSubject string: '%s'\n\n",
*(two_byte_subject->ToCString()));
Either comment it back in or remove it.

http://codereview.chromium.org/12473/diff/1/3
File src/jsregexp.h (right):

http://codereview.chromium.org/12473/diff/1/3#newcode745
Line 745: explicit Analysis(bool case_independent)
We're somewhat inconsistent in what we call this: case_independent, ignore_case,
etc.  We should settle on just one and I vote for ignore_case.

Powered by Google App Engine
This is Rietveld 408576698