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

[Alexandre Julliard]

Dmitry Timoshkov  writes:
> Alexandre Julliard  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?