[Mono-bugs] [Bug 77335][Maj] Changed - Timer events are not delivered by Application.DoEvents

[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 peter at novonyx.com.

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

--- shadow/77335	2006-01-23 20:32:10.000000000 -0500
+++ shadow/77335.tmp.7323	2006-01-25 20:48:46.000000000 -0500
@@ -1,13 +1,13 @@
 Bug#: 77335
 Product: Mono: Class Libraries
 Version: unspecified
 OS: unknown
 OS Details: Windows XP
-Status: NEW   
-Resolution: 
+Status: RESOLVED   
+Resolution: FIXED
 Severity: Unknown
 Priority: Major
 Component: Windows.Forms
 AssignedTo: peter at novonyx.com                            
 ReportedBy: don at edvalson.net               
 QAContact: mono-bugs at ximian.com
@@ -53,6 +53,18 @@
 Modified XplatUIWin32.cs that fixes this bug.
 
 
 ------- Additional Comments From don at edvalson.net  2006-01-23 20:32 -------
 I have attached a version of XplatUIWin32.cs that seems to solve the 
 problem by having DoEvents call the same logic as GetMessage
+
+------- Additional Comments From peter at novonyx.com  2006-01-25 20:48 -------
+Fixed in svn r56087. Thanks for your patch. I took a slightly 
+different approach in fixing, though, keeping to avoid the double 
+loop in DoEvents (which would never access the private queue again 
+once it was worked off), and allowing us to have a backend for a 
+PeekMessage driver method.
+You don't have to attach whole files as patches, btw, you can simply 
+attach the output of 'svn diff <file>' as a patch. Makes it easier to 
+see what's been changed. If you're not working from svn, you can do 
+a 'diff -u <origfile> <newfile>'. Either way, many thanks for all 
+your useful bug reports!