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 [EMAIL PROTECTED]

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

--- shadow/81627        2007-07-19 12:55:16.000000000 -0400
+++ shadow/81627.tmp.4571       2007-07-19 16:30:16.000000000 -0400
@@ -240,6 +240,17 @@
 the C side of the changes 
 
 ------- Additional Comments From [EMAIL PROTECTED]  2007-07-19 12:55 -------
 So far I have only implemented the managed side for MS.net's
 behaviour. I only then found out that the runtime code needs to be
 changed too (although I suspected that from the beginning).
+
+------- Additional Comments From [EMAIL PROTECTED]  2007-07-19 16:30 -------
+Juraj, your patch looks fine for the managed side, but there are a few
+issues:
+
+-it should clone the returned array in GetInvocationList().
+-it should make sure that only the first element have an array, this
+means that on CombineImpl it must clone the first element of the
+follow Delegate if it has an array.
+
+Besides that, it looks great :)
_______________________________________________
mono-bugs maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-bugs

Reply via email to