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

Issue 94002: Fix for Issue 317 - bug in string.replace(string, "$foo"). (Closed)

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

Description

Fix for Issue 317 - bug in string.replace(string, "$foo"). Fix for Issue 317. Replace string with something containing dollar now works.

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+42 lines, -0 lines) Patch
M src/string.js View 1 chunk +2 lines, -0 lines 0 comments Download
A test/mjsunit/regress/regress-317.js View 1 chunk +31 lines, -0 lines 1 comment Download
M test/mjsunit/string-replace.js View 1 chunk +9 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Lasse Reichstein
Review, please.
11 years, 8 months ago (2009-04-22 11:34:21 UTC) #1
Lasse Reichstein
Review, please.
11 years, 8 months ago (2009-04-22 11:34:25 UTC) #2
Mads Ager (chromium)
11 years, 8 months ago (2009-04-22 11:37:12 UTC) #3
LGTM

http://codereview.chromium.org/94002/diff/1/3
File test/mjsunit/regress/regress-317.js (right):

http://codereview.chromium.org/94002/diff/1/3#newcode30
Line 30: assertEquals("a$ec", "abc".replace("b", "$e"), "$e isn't meaningfull");
meaningful

Powered by Google App Engine
This is Rietveld 408576698