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

[Dmitry Timoshkov]

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.

-- 
Dmitry.