Re: [PATCH 0/5] MR3492: clean up user32:msg tests

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/user32/tests/msg.c:
>                  if ((expected->lParam ^ actual->lParam) & ~expected->lp_mask) dump++;
>              }
>          }
> -
>    if ((expected->flags & optional) &&
> -
>            ((expected->flags ^ actual->flags) & (defwinproc|parent)))
> -
>        {

We were previously reaching this case when the messages or flags differ, without printing an error. With this change, I think we'd be getting a test failure from messages_equal before checking the optional / defwinproc flag?