matcher: fixed status file, formatting, tweaks to readme

lib/src/pretty_print.dart

 // Copyright (c) 2013, 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.
 
 library matcher.pretty_print;
 
 import 'description.dart';
 import 'interfaces.dart';
 
 /// Returns a pretty-printed representation of [object].
@@ skipping 62 matching lines @@
         return singleLine;
       }
 
       // Otherwise, print each key/value pair on its own line.
       return "{\n" + strings.map((string) {
         return _indent(indent + 2) + string;
       }).join(",\n") + "\n" + _indent(indent) + "}";
     } else if (object is String) {
       // Escape strings and print each line on its own line.
       var lines = object.split("\n");
-      return "'" + lines.map(_escapeString)
-          .join("\\n'\n${_indent(indent + 2)}'") + "'";
+      return "'" +
+          lines.map(_escapeString).join("\\n'\n${_indent(indent + 2)}'") +
+          "'";
     } else {
       var value = object.toString().replaceAll("\n", _indent(indent) + "\n");
       var defaultToString = value.startsWith("Instance of ");
 
       // If this is the top-level call to [prettyPrint], wrap the value on angle
       // brackets to set it apart visually.
       if (top) value = "<$value>";
 
       // Print the type of objects with custom [toString] methods. Primitive
       // objects and objects that don't implement a custom [toString] don't need
       // to have their types printed.
-      if (object is num || object is bool || object is Function ||
-          object == null || defaultToString) {
+      if (object is num ||
+          object is bool ||
+          object is Function ||
+          object == null ||
+          defaultToString) {
         return value;
       } else {
         return "${_typeName(object)}:$value";
       }
     }
   }
 
   return _prettyPrint(object, 0, new Set(), true);
 }
 
@@ skipping 14 matching lines @@
     return "?";
   }
 }
 
 /// Returns [source] with any control characters replaced by their escape
 /// sequences.
 ///
 /// This doesn't add quotes to the string, but it does escape single quote
 /// characters so that single quotes can be applied externally.
 String _escapeString(String source) =>
-  source.split("").map(_escapeChar).join("");
+    source.split("").map(_escapeChar).join("");
 
 /// Return the escaped form of a character [ch].
 String _escapeChar(String ch) {
-  if (ch == "'")
-    return "\\'";
-  else if (ch == '\n')
-    return '\\n';
-  else if (ch == '\r')
-    return '\\r';
-  else if (ch == '\t')
-    return '\\t';
-  else
-    return ch;
+  if (ch == "'") return "\\'";
+  else if (ch == '\n') return '\\n';

[Siggi Cherem (dart-lang), 2015/01/14 20:30:09]

we should be able to remove all the `else`s here

[kevmoo, 2015/01/14 22:28:45]

Yup. Actually used a case statement here. Seems a bit more canonical.

+  else if (ch == '\r') return '\\r';
+  else if (ch == '\t') return '\\t';
+  else return ch;
 }
-