[Mono-bugs] [Bug 76931][Blo] Changed - Return value of ISerializationSurrogate.SetObjectData() ignored

[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 rharinath at novell.com.

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

--- shadow/76931	2005-12-22 09:43:41.000000000 -0500
+++ shadow/76931.tmp.14451	2005-12-22 09:51:33.000000000 -0500
@@ -82,6 +82,17 @@
 prevents the transitive update of ListItem.
 
 ------- Additional Comments From rharinath at novell.com  2005-12-22 09:43 -------
 Created an attachment (id=16245)
 proposed fix
 
+
+------- Additional Comments From rharinath at novell.com  2005-12-22 09:51 -------
+The patch above fixes the regression that was uncovered.
+
+Basically, the old test only worked if the parent class of a value
+field was also [Serializable] and thus had a non-null MemberInfo for
+that field.
+
+If necessary, I think we can cache the fact that the ObjectInstance is
+a value type in some flag -- though, given the number of icalls
+involved in deserialization, I don't expect this to slow down things much.