Re: [PATCH 1/2] user32: Add more tests for SetWindowPos.

[Alexandre Julliard]

"Lei Zhang"  writes:
> +static UINT sequence_contains(UINT msg)
> +{
> - static const struct message end_of_sequence = { 0, 0, 0, 0 };
> - const struct message *actual;
> - UINT num = 0;
> -
> - add_message(&end_of_sequence);
> - actual = sequence;
> -
> - while (actual->message)
> - {
> -
>    if (actual->message == msg) num++;
> -
>    actual++;
> - }
> - return num;
>
> +}

The fix looks good now, but you should really test the message sequences
instead of just counting a single message. We have already all the
necessary mechanisms for this.