[Mono-bugs] [Bug 81627][Nor] Changed - FileSystemWatcher RaiseEvent Causes Huge Heap

[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 kumpera at gmail.com.

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 juraj at hotfeet.ch  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 kumpera at gmail.com  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 :)