[Mono-bugs] [Bug 25433][Nor] Changed - mcs chokes on NullRefException when assigning event handlers

[bugzilla-daemon@rocky.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 miguel@ximian.com.

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

--- shadow/25433	Thu Jun 13 15:26:47 2002
+++ shadow/25433.tmp.9158	Sat Jun 22 19:26:50 2002
@@ -70,6 +70,19 @@
 I agree fully with Paolo's assessment of the problem - perhaps I 
 need to revamp the way I deal with events and their transformation 
 into fields. The problem is that GetFieldFromEvent () is really a 
 workaround more than a neat approach to the problem
 
 I am going to tackle this soon.
+
+------- Additional Comments From miguel@ximian.com  2002-06-22 19:26 -------
+Any updates on this?  I figured that if we are turning accesses to
+events to fields then we should basically just do a
+RegisterEventField, and use this to look things up (this is the source
+of the problem).
+
+The current implementation uses TypeContainers which will not work
+once you hit a Type. 
+
+(btw, what happens if you have an add/remove handler in that case, the
+current code just turns that into a field access, seems wrong to me)
+