On Fri, Nov 6, 2009 at 9:18 AM, Robert Jordan <span dir="ltr">&lt;<a href="mailto:robertj@gmx.net">robertj@gmx.net</a>&gt;</span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">Jonathan Hseu wrote:<br>
&gt; Caveats:<br>
&gt;<br>
&gt; This new DateTime is binary incompatible (again with BinaryFormatter) with<br>
&gt; older versions of Mono.<br>
<br>
</div>That&#39;s the reason why the fixes attached to<br>
<div class="im"><br>
<a href="https://bugzilla.novell.com/show_bug.cgi?id=325067" target="_blank">https://bugzilla.novell.com/show_bug.cgi?id=325067</a><br>
<br>
</div>were not accepted.<br>
<br>
Robert<br></blockquote><div><br></div><div>Going through the bug logs, it seems like the patch was first applied but then later removed because it was still incompatible with .NET&#39;s version, not because it was incompatible with older versions of mono.</div>
<div><br></div><div>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.</div>
<div><br></div><div>Also, given that mono&#39;s DateTime currently has an extra 8 byte overhead over .NET, shouldn&#39;t this issue be looked at again?</div><div><br></div><div><br></div><div>Jonathan Hseu</div></div>