<html><body><div style="color:#000; background-color:#fff; font-family:times new roman, new york, times, serif;font-size:12pt"><div><span>Maybe you're running an older version of my unit test, before I added code that ensured the event-handler was being called.  I've attached a fresh copy of patch #1.</span></div><div><span><br></span></div><div><span>Enclosed is a reworked version of patch #4, one whose behavior has been tested against .NET.  Apply the unit test patch, see that you get a unit test failure, then apply my patch, and see that the unit test no longer fails.  Also enclosed is a ComboBoxTest project containing the unit test, so you can run it under .NET easily.</span></div><div><span><br></span></div><div><span>Also, I noticed that bringing up a dialog box before calling Application.Run() tends to make Mono freak out.  Maybe that's the problem with your DgwTest project?  The enclosed ComboBoxTest project waits until
 the Form's Load event to bring up dialog boxes.</span></div><div><span><br></span></div><div><span>I will go and redo patch #3 so that it works under .NET, and try to come up with unit tests for patches 2 and 7.  Those require user interaction...if I can figure out how to fake a mouse click, I should be able to do this.  Looks like the accepted method is to create a subclass and then call OnMouseClick() directly.</span></div><div><span><br></span></div><div><span>Steven Boswell</span></div><div><br></div>  <div style="font-size: 12pt; font-family: 'times new roman', 'new york', times, serif; "> <div style="font-size: 12pt; font-family: 'times new roman', 'new york', times, serif; "> <div dir="ltr"> <font size="2" face="Arial"> <hr size="1">  <b><span style="font-weight:bold;">From:</span></b> Stifu <stifu@free.fr><br> <b><span style="font-weight: bold;">To:</span></b> mono-devel-list@lists.ximian.com <br> <b><span style="font-weight:
 bold;">Sent:</span></b> Saturday, June 9, 2012 9:34 AM<br> <b><span style="font-weight: bold;">Subject:</span></b> Re: [Mono-dev] Patches for mono-winforms<br> </font> </div> <br>
"Um...you're not supposed to get the "Event handler!" message-box with an<br>unpatched Mono.  The patch's main purpose is to make sure that event handler<br>gets called.  Or do I misunderstand what you're saying?"<br><br>Then isn't the test wrong? If the event handler doesn't get executed, then<br>the unit test still passes. I just tried commenting the event handler and<br>run the test with .NET, and it passes.<br><br>By the way, I think there may have been issues with patch 3... Now the<br>concerned 2 tests (SelectedColumnsTest and SelectedRowsTest) don't pass<br>anymore with .NET, it seems.<br><br>I get: 1-14<br>  Expected: 0<br>  But was:  2<br><br>Can you confirm? I did stumble across this error while reviewing your patch,<br>but then I thought it ended up passing. I must have made a mistake<br>somewhere.<br><br>I didn't actually find your GitHub account, just referenced your e-mail<br>address and that's it. :p<br>C# Qt
 bindings is a cool project. I was gonna mention CXXI, but you already<br>know about it.<br><br><br>Steven Boswell II wrote<br>> <br>> Um...you're not supposed to get the "Event handler!" message-box with an<br>> unpatched Mono.  The patch's main purpose is to make sure that event<br>> handler gets called.  Or do I misunderstand what you're saying?<br>> <br>> Thanks for committing my other patch.  I had totally forgotten about my<br>> account on GitHub.  Kudos for finding it. :-)<br>> <br>> Steven Boswell<br>> <br>> <br>> ________________________________<br>>  From: Stifu &lt;stifu@&gt;<br>> To: mono-devel-list@.ximian <br>> Sent: Saturday, June 9, 2012 12:03 AM<br>> Subject: Re: [Mono-dev] Patches for mono-winforms<br>>  <br>> By "ran fine", do you mean you do get the "Event handler!" message box?<br>> Because I don't, with Mono (unpatched).<br>> <br>>
 Patch 8 pushed, slightly reformatted<br>> (<a href="https://github.com/mono/mono/commit/641ae7303320c2bad8ff83f9b5f9fc0d285b6b0e" target="_blank">https://github.com/mono/mono/commit/641ae7303320c2bad8ff83f9b5f9fc0d285b6b0e</a>).<br>> Thanks.<br>> _______________________________________________<br>> Mono-devel-list mailing list<br>> Mono-devel-list@.ximian<br>> <a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a><br>> <br><br>--<br>View this message in context: http://mono.1490590.n4.nabble.com/Patches-for-mono-winforms-tp4649620p4649820.html<br>Sent from the Mono - Dev mailing list archive at <a target="_blank" href="http://Nabble.com">Nabble.com</a>.<br>_______________________________________________<br>Mono-devel-list mailing list<br><a ymailto="mailto:Mono-devel-list@lists.ximian.com"
 href="mailto:Mono-devel-list@lists.ximian.com">Mono-devel-list@lists.ximian.com</a><br>http://lists.ximian.com/mailman/listinfo/mono-devel-list<br><br><br> </div> </div>  </div></body></html>