Fix VariableDefinitions.endToken for formal parameters.

sdk/lib/_internal/compiler/implementation/tree/unparser.dart

Index: sdk/lib/_internal/compiler/implementation/tree/unparser.dart
diff --git a/sdk/lib/_internal/compiler/implementation/tree/unparser.dart b/sdk/lib/_internal/compiler/implementation/tree/unparser.dart
index 4a0e6e74903cba6791ec3fde71abea2db436a650..463ce72dbb6c40f1a7c2c6a1c0b2136abb37415f 100644
--- a/sdk/lib/_internal/compiler/implementation/tree/unparser.dart
+++ b/sdk/lib/_internal/compiler/implementation/tree/unparser.dart
@@ -362,7 +362,9 @@ class Unparser implements Visitor {
       sb.add(' ');
     }
     visit(node.definitions);
-    if (node.endToken.value == const SourceString(';')) {
+
+    if (node.endToken != null &&
+        node.endToken.value == const SourceString(';')) {

[ahe, 2013/01/23 11:26:36]

I think you only need to test for "node.endToken != null" now. In other words,
if it has an endToken, it should be printed.

However, don't we already have the same logic for NodeList?

[Johnni Winther, 2013/01/28 12:23:24]

We can remove endToken completely since node.definitions handles it.

       add(node.endToken.value);
     }
   }