489288 - Wince WaitMessage is wrong

Status: RESOLVED WONTFIX

(Core :: Widget: Win32, defect)

Description

[Doug Turner (:dougt)]

in the simulation of WaitMessage(), we should use QS_POSTMESSAGE instead of QS_ALLEVENTS.

According to the documentation [1], WaitMessage returns control to the calling thread when "a new message is placed in the thread's message queue".  This is consistent with the QS_POSTMESSAGE parameter [2].

[1] http://msdn.microsoft.com/en-us/library/ms644956%28VS.85%29.aspx  (w32)
[2] http://msdn.microsoft.com/en-us/library/ms961244.aspx  (wince)


Right now, what we have is going to be return more often than needed.

Comment 1

[Doug Turner (:dougt)]

Attachment: patch v.1

Comment 2

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Hrm, are you sure about this?  QS_POSTMESSAGE is basically going to ignore most input, paint etc. messages, which I'm pretty sure WaitMessage() won't.

Comment 3

[Doug Turner (:dougt)]

yeah, weird huh.  we tried this patch, and it is terrible;  dropping event.