Fix issue with optional named parameters.

tests/language/optional_named_parameters_test.dart

Index: tests/language/optional_named_parameters_test.dart
===================================================================
--- tests/language/optional_named_parameters_test.dart	(revision 12210)
+++ tests/language/optional_named_parameters_test.dart	(working copy)
@@ -62,28 +62,28 @@
     Expect.equals(10, np.f22(10));
     Expect.equals(20, F10());
     Expect.equals(20, np.f21());
-    Expect.equals(20, F10(20));
-    Expect.equals(20, np.f21(20));
+    Expect.equals(20, F10(20));  /// 01: runtime error

[hausner, 2012/09/11 17:37:53]

The test has to run with --reject_named_argument_as_positional to trigger these
runtime errors, right? Don't you have to include
VMOptions=--reject_named_argument_as_positional at the top of this file?

[regis, 2012/09/11 18:02:35]

No, we always want to detect this error. The flag only relaxes the rules when
passing named arguments to positional parameters in [], but not when passing
positional arguments to named parameters in {}. This is actually what this cl
fixes.

+    Expect.equals(20, np.f21(20));  /// 02: runtime error
     Expect.equals(20, F10(b:20));
     Expect.equals(20, np.f21(b:20));
     Expect.equals(1020, F21(10));
     Expect.equals(1020, np.f32(10));
-    Expect.equals(1025, F21(10, 25));
-    Expect.equals(1025, np.f32(10, 25));
+    Expect.equals(1025, F21(10, 25));  /// 03: runtime error
+    Expect.equals(1025, np.f32(10, 25));  /// 04: runtime error
     Expect.equals(1025, F21(10, b:25));
     Expect.equals(1025, np.f32(10, b:25));
     Expect.equals(102030, F31(10));
     Expect.equals(102030, np.f42(10));
-    Expect.equals(102530, F31(10, 25));
-    Expect.equals(102530, np.f42(10, 25));
+    Expect.equals(102530, F31(10, 25));  /// 05: runtime error
+    Expect.equals(102530, np.f42(10, 25));  /// 06: runtime error
     Expect.equals(102530, F31(10, b:25));
     Expect.equals(102530, np.f42(10, b:25));
     Expect.equals(102035, F31(10, c:35));
     Expect.equals(102035, np.f42(10, c:35));
     Expect.equals(102535, F31(10, b:25, c:35));
     Expect.equals(102535, np.f42(10, b:25, c:35));
-    Expect.equals(102535, F31(10, 25, c:35));
-    Expect.equals(102535, np.f42(10, 25, c:35));
+    Expect.equals(102535, F31(10, 25, c:35));  /// 07: runtime error
+    Expect.equals(102535, np.f42(10, 25, c:35));  /// 08: runtime error
     Expect.equals(102535, F31(10, c:35, b:25));
     Expect.equals(102535, np.f42(10, c:35, b:25));
     Expect.equals(10200040, F41(10));
@@ -91,7 +91,7 @@
     Expect.equals(10203540, F41(10, c:35));
     Expect.equals(10203540, np.f52(10, c:35));
     Expect.equals(10250045, F41(10, d:45, b:25));
-    Expect.equals(10250045, F41(10, 25, d:45));
+    Expect.equals(10250045, F41(10, 25, d:45));  /// 09: runtime error
     Expect.equals(10250045, np.f52(10, d:45, b:25));
     Expect.equals(10253545, F41(10, d:45, c:35, b:25));
     Expect.equals(10253545, np.f52(10, d:45, c:35, b:25));