[Mono-dev] [PATCH] Make DateTime binary compatible with .NET

[Leszek Ciesielski]

On Fri, Nov 6, 2009 at 4:32 PM, Jonathan Hseu <vomjom at vomjom.net> wrote:
> On Fri, Nov 6, 2009 at 9:18 AM, Robert Jordan <robertj at gmx.net> wrote:
>>
>> Jonathan Hseu wrote:
>> > Caveats:
>> >
>> > This new DateTime is binary incompatible (again with BinaryFormatter)
>> > with
>> > older versions of Mono.
>>
>> That's the reason why the fixes attached to
>>
>> https://bugzilla.novell.com/show_bug.cgi?id=325067
>>
>> were not accepted.
>>
>> Robert
>
> Going through the bug logs, it seems like the patch was first applied but
> then later removed because it was still incompatible with .NET's version,
> not because it was incompatible with older versions of mono.
> My patch is completely different than the one in the bug report.  Mine
> _only_ affects the DateTime.cs file whereas the one in the bug report
> changed the serialization code.  My patch basically introduces the same
> field names and types that are also in .NET so that it is guaranteed to be
> compatible.
> Also, given that mono's DateTime currently has an extra 8 byte overhead over
> .NET, shouldn't this issue be looked at again?
>

Would the patch get accepted if it still was able to deserialize old
(i.e. currently used) Mono DateTime format, while only outputting the
.Net compatible one? This is an incompatibility that has to get fixed
eventually.