[Mono-bugs] [Bug 81101][Nor] Changed - [Patch] Control.WindowTarget implementation

[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 toshok at ximian.com.

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

--- shadow/81101	2007-03-10 10:41:25.000000000 -0500
+++ shadow/81101.tmp.23041	2007-03-15 07:55:48.000000000 -0500
@@ -1,13 +1,13 @@
 Bug#: 81101
 Product: Mono: Class Libraries
 Version: 1.2
 OS: unknown
 OS Details: 
-Status: NEW   
-Resolution: 
+Status: RESOLVED   
+Resolution: FIXED
 Severity: Unknown
 Priority: Normal
 Component: Windows.Forms
 AssignedTo: toshok at ximian.com                            
 ReportedBy: contact at i-nz.net               
 QAContact: mono-bugs at ximian.com
@@ -47,6 +47,20 @@
 doesn't have special constrains.
 
 ------- Additional Comments From contact at i-nz.net  2007-03-10 10:41 -------
 Created an attachment (id=18873)
 Patch + Test + ChangeLog entry
 
+
+------- Additional Comments From toshok at ximian.com  2007-03-15 07:55 -------
+for future reference, when writing unit tests please run them against
+.net - they're both regression tests and compliance tests.  Your unit
+tests failed against windows (WindowTarget2, specifically.)
+
+I added another test of the setter, and have changed
+Control.set_Target so that it works as windows does.
+
+One thing that still *does* need testing is whether or not setting
+Control.Target to null causes an NRE on windows when the control
+receives a message.  It will on mono.
+
+marking fixed.