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

Issue 10411048: Fix issue 2756 where '$' has as special meaning in Javascript String.replace. (Closed)

Created:
8 years, 7 months ago by ngeoffray
Modified:
8 years, 7 months ago
Reviewers:
ahe, floitsch
CC:
reviews_dartlang.org, kasperl, karlklose, Lasse Reichstein Nielsen
Visibility:
Public.

Description

Fix issue 2756 where '$' has as special meaning in Javascript String.replace. Committed: https://code.google.com/p/dart/source/detail?r=7787

Patch Set 1 #

Total comments: 2

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+30 lines, -5 lines) Patch
M lib/compiler/implementation/lib/string_helper.dart View 1 3 chunks +13 lines, -5 lines 0 comments Download
A tests/corelib/string_replace_dollar_test.dart View 1 1 chunk +17 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
ngeoffray
8 years, 7 months ago (2012-05-21 08:26:52 UTC) #1
floitsch
LGTM. https://chromiumcodereview.appspot.com/10411048/diff/1/lib/compiler/implementation/lib/string_helper.dart File lib/compiler/implementation/lib/string_helper.dart (right): https://chromiumcodereview.appspot.com/10411048/diff/1/lib/compiler/implementation/lib/string_helper.dart#newcode93 lib/compiler/implementation/lib/string_helper.dart:93: to = JS('String', @"#.replace('\$', '$$$$')", to); Add comment ...
8 years, 7 months ago (2012-05-21 08:49:15 UTC) #2
ngeoffray
Thanks Florian. I have also updated all the places where we use JS String.replace. https://chromiumcodereview.appspot.com/10411048/diff/1/lib/compiler/implementation/lib/string_helper.dart ...
8 years, 7 months ago (2012-05-21 10:14:57 UTC) #3
ahe
8 years, 7 months ago (2012-05-21 10:20:06 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld 408576698