Support replaceAll for regexps.

frog/lib/string_implementation.dart

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 //String.prototype.get$length = function() {
 //  return this.length;
 //}
 
 // TODO(jimhug): Unify with code from compiler/lib/implementation.
 class StringImplementation implements String native "String" {
@@ skipping 27 matching lines @@
 
   String trim() native;
 
   // TODO(jmesserly): should support pattern too.
   bool contains(Pattern pattern, [int startIndex]) native
     "return this.indexOf(pattern, startIndex) >= 0;";
 
   String _replaceFirst(String from, String to) native
     "return this.replace(from, to);";
 
-  String _replaceFirstRegExp(RegExp from, String to) native
+  String _replaceRegExp(RegExp from, String to) native
     "return this.replace(from.re, to);";
 
   String replaceFirst(Pattern from, String to) {
     if (from is String) return _replaceFirst(from, to);
-    if (from is RegExp) return _replaceFirstRegExp(from, to);
-    for (match in from.allMatches(this)) {
+    if (from is RegExp) return _replaceRegExp(from, to);
+    for (var match in from.allMatches(this)) {

[Jennifer Messerly, 2011/12/14 20:36:56]

Thanks for making this more consistent with the rest of the frog code!

Interesting style question though: I've noticed that I'm using the "for (var
match in ...)" style. I'm not sure why I was doing that, other than old C#
habits. I wonder if we should switch to the "for (match in ..." style? Anyway
not something to worry about in the short term :)

       // We just care about the first match
       return substring(0, match.start()) + to + substring(match.end());
     }
   }
 
-  String replaceAll(Pattern from, String to) native @"""
-if (typeof(from) == 'string' || from instanceof String) {
-  from = new RegExp(from.replace(/[-[\]{}()*+?.,\\^$|#\s]/g, "\\$&"), 'g');
-  to = to.replace(/\$/g, '$$$$'); // Escape sequences are fun!
-}
+  String _replaceAll(String from, String to) native @"""
+from = new RegExp(from.replace(/[-[\]{}()*+?.,\\^$|#\s]/g, "\\$&"), 'g');
+to = to.replace(/\$/g, '$$$$'); // Escape sequences are fun!
 return this.replace(from, to);""";
 
+  String replaceAll(Pattern from, String to) {
+    if (from is String) return _replaceAll(from, to);
+    if (from is RegExp) return _replaceRegExp(from.dynamic._global, to);
+    var buffer = new StringBuffer();
+    var lastMatchEnd = 0;
+    for (var match in from.allMatches(this)) {
+      buffer.add(substring(lastMatchEnd, match.start()));
+      buffer.add(to);
+      lastMatchEnd = match.end();
+    }
+    buffer.add(substring(lastMatchEnd));
+  }
+
   List<String> split(Pattern pattern) native;
 
   /*
   Iterable<Match> allMatches(String str) {
     List<Match> result = [];
     if (this.isEmpty()) return result;
     int length = this.length;
 
     int ix = 0;
     while (ix < str.length) {
@@ skipping 65 matching lines @@
       result.add(group(g));
     }
     return result;
   }
 
   final int _start;
   final String str;
   final String pattern;
 }
 */