[Mono-bugs] [Bug 82400][Nor] Changed - [PATCH] DateTime binary serialization incompatible with MS.NET 2.0

[bugzilla-daemon at bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by jaebird at gmail.com.

http://bugzilla.ximian.com/show_bug.cgi?id=82400

--- shadow/82400	2007-08-21 14:47:56.000000000 -0400
+++ shadow/82400.tmp.4709	2007-08-21 15:11:55.000000000 -0400
@@ -182,6 +182,20 @@
 The ISerializable implementation is still needed for custom formatters.
 
 
 Any reasons why you didn't use DateTime.ToBinary () and
 DateTime.FromBinary ()?
 
+
+------- Additional Comments From jaebird at gmail.com  2007-08-21 15:11 -------
+ToBinary() and FromBinary() would also have worked here, and should be
+used instead. Since I was attacking the problem from the data logs of
+the network, I didn't notice those methods used the same method of
+mashing.
+
+Also for the record:
+I would not say that this fixes binary serialization. If you do a
+binary serialization with BinaryFormatter of DateTime (ie to a file)
+it must do it the way Lionel's patch does it in the custom formatter.
+
+This change is smaller and fixes the remoting problem we were seeing.
+I'm leaving the other issue alone for now.