Add optional writeAll separator argument.

sdk/lib/core/string_buffer.dart

Index: sdk/lib/core/string_buffer.dart
diff --git a/sdk/lib/core/string_buffer.dart b/sdk/lib/core/string_buffer.dart
index 50bb51c4249c67031d4d2ce6bb674eac1a8d1c2c..45462195b861da856f28fd337bef1557b50ce878 100644
--- a/sdk/lib/core/string_buffer.dart
+++ b/sdk/lib/core/string_buffer.dart
@@ -29,8 +29,16 @@ class StringBuffer implements StringSink {
   /// Adds the string representation of [charCode] to the buffer.
   external void writeCharCode(int charCode);
 
-  void writeAll(Iterable objects) {
-    for (Object obj in objects) write(obj);
+  void writeAll(Iterable objects, [String separator = ""]) {
+    bool isFirst = true;
+    for (Object obj in objects) {
+      if (isFirst) {
+        isFirst = false;
+      } else {
+        if (separator != "") write(separator);

[sra1, 2013/03/22 21:19:46]

Can we avoid this repeated test in the loop?
Or at least fold it with isFirst:

if (separator == "") separator = null;
String nextSeparator = null;
for (var obj in objects) {
  if (nextSeparator != null) write(nextSeparator);
  nextSeparator = separator;
  write(object);
}

[floitsch, 2013/03/26 18:30:23]

Done.

[Lasse Reichstein Nielsen, 2013/04/03 10:35:52]

If speed is of the essence, I'd prefer:
 void writeAll(Iterable objects, [String separator = ""]) {
   if (separator.isEmpty) {
     for (Object object in objects) {
       write(object);
     }
   } else {
     Iterator iterator = objects.iterator();
     if (!iterator.moveNext()) return;
     write(iterator.current);
     while (iterator.moveNext()) {
       write(separator);
       write(iterator.current);
     }
   }
 }

Or, at least initialize nextSeparator to "" and drop the test completely.

[floitsch, 2013/04/04 22:36:36]

I didn't use Stephen's suggestion but had a code that is pretty similar to
yours. Assuming as "done", but feel free to verify.

+      }
+      write(obj);
+    }
   }
 
   void writeln([Object obj = ""]) {