user32: After handling an internal message give a chance to real message to arrive.

[Alexandre Julliard]

Dmitry Timoshkov <dmitry at baikal.ru> writes:

> Alexandre Julliard <julliard at winehq.org> wrote:
>
>> >> > @@ -2931,7 +2931,10 @@ static BOOL peek_message( MSG *msg, HWND hwnd, UINT first, UINT last, UINT flags
>> >> >                  else
>> >> >                      peek_message( msg, info.msg.hwnd, info.msg.message,
>> >> >                                    info.msg.message, flags | PM_REMOVE, changed_mask );
>> >> > -                continue;
>> >> > +
>> >> > +                /* give a chance to real message to arrive */
>> >> > +                HeapFree( GetProcessHeap(), 0, buffer );
>> >> > +                return FALSE;
>> >> 
>> >> Why do you need to leave the function instead of checking for further
>> >> messages? What is the issue you are seeing?
>> >
>> > If you mean the only quoted piece of code (MSG_POSTED case) then only
>> > reason of the change is to make it consistent with the sent message case.
>> 
>> I don't really see any reason to make it consistent, but I'd like to
>> know why you need the other one too.
>
> The other one is needed to make the test pass for the sent message case.
> If you have a suggestion how fix it differently please let me know.

I may have one, once I understand what you are trying to fix. What is
the actual problem that prompted you to write the tests and the fix?

-- 
Alexandre Julliard
julliard at winehq.org