Normalization of patch signatures.

lib/core/date.dart

Index: lib/core/date.dart
diff --git a/lib/core/date.dart b/lib/core/date.dart
index e36a8ba57a21edeb0ebfad583dff903c8b4a0bd7..0db2132db71b2a1d2dc4809e69b816aef9973d77 100644
--- a/lib/core/date.dart
+++ b/lib/core/date.dart
@@ -48,13 +48,13 @@ interface Date extends Comparable, Hashable default DateImplementation {
   // TODO(floitsch): the spec allows default values in interfaces, but our
   // tools don't yet. Eventually we want to have default values here.
   Date(int year,
-       [int month,
-        int day,
-        int hour,
-        int minute,
-        int second,
-        int millisecond,
-        bool isUtc]);
+       [int month = 1,

[Mads Ager (google), 2012/09/17 05:52:56]

Are default values allowed in interfaces now? They used to not be. Do you need
to change the interface here? This is not what is being patched.
DateImplementation is what is being patched (at this point).

[Lasse Reichstein Nielsen, 2012/09/17 11:32:43]

It's allowed since v. 0.09.
But it's true that it's not related to patching.

I like the default values. We will change the interface to an abstract class
Real Soon Now, and probably to a normal factory constructor to begin with. I.e.,
personally I don't mind that this is changed, even if it's not needed.

[Mads Ager (google), 2012/09/17 11:37:26]

Great, now that it is allowed I'm actually strongly in favor of this change. Not
having the default values in the interface is a mess. Thanks Lasse.

[Johnni Winther, 2012/09/19 09:03:30]

It seems to be allowed now and the TODO above wants it too. But you are right,
it is only needed in DateImplementation.

+        int day = 1,
+        int hour = 0,
+        int minute = 0,
+        int second = 0,
+        int millisecond = 0,
+        bool isUtc = false]);
 
   /**
    * Constructs a new [Date] instance with current date time value in the