Handle backslashes and newlines when escaping strings.

dart/lib/core/errors.dart

 // Copyright (c) 2012, 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.
 
 class Error {
   const Error();
 }
 
 class AssertionError implements Error {
 }
@@ skipping 63 matching lines @@
       }
       String formalParameters = sb.toString();
       return "NoSuchMethodError: incorrect number of arguments passed to "
           "method named '$_functionName'\n"
           "Receiver: ${safeToString(_receiver)}\n"
           "Tried calling: $_functionName($actualParameters)\n"
           "Found: $_functionName($formalParameters)";
     }
   }
 
-  external static String safeToString(Object object);
+  static String safeToString(Object object) {
+    if (object is int || object is double || object is bool || null == object) {

[Lasse Reichstein Nielsen, 2012/09/19 12:14:53]

The first two can be condensed to 'object is num'.

[ahe, 2012/09/19 15:59:45]

No. I'll leave it as an exercise to the reader to prove why. Hint: sqrt(-1).

+      return object.toString();
+    }
+    if (object is String) {
+      // TODO(ahe): Remove backslash when http://dartbug.com/4995 is fixed.
+      const backslash = '\\';
+      String escaped = object
+        .replaceAll('$backslash', '$backslash$backslash')
+        .replaceAll('\n', '${backslash}n')

[Lasse Reichstein Nielsen, 2012/09/19 12:14:53]

Ditto for "\r", which is also a "newline" in Dart. (Good thing you don't need to
handle U+2028 and U+2029 too).

[ahe, 2012/09/19 15:59:45]

Done. Also extended the test.

+        .replaceAll('"',  '$backslash"');

[Lasse Reichstein Nielsen, 2012/09/19 12:14:53]

We SO need a computed replace to avoid these sequential replacements.

+      return '"$escaped"';
+    }
+    return _objectToString(object);
+  }
+
+  external static _objectToString(Object object);
 }