Re: user32: Partially implement FlashWindowEx with tests

[Dmitry Timoshkov]

André Hentschel  wrote:
> - if (!pfwi->hwnd || pfwi->cbSize < sizeof(FLASHWINFO))
> - {
> -
>    SetLastError(ERROR_INVALID_PARAMETER);
> -
>    return FALSE;
> - }

I don't see in your tests the case for 'cbSize > sizeof(FLASHWINFO)'.
Also using !IsWindow() instead of !hwnd would fix one more test case.

user32: Partially implement FlashWindowEx with tests

[André Hentschel]

Am 22.05.2011 09:58, schrieb Dmitry Timoshkov:
> André Hentschel  wrote:
> > - if (!pfwi->hwnd || pfwi->cbSize < sizeof(FLASHWINFO))
> > - {
> > -
> >    SetLastError(ERROR_INVALID_PARAMETER);
> > -
> >    return FALSE;
> > - }
>
> I don't see in your tests the case for 'cbSize > sizeof(FLASHWINFO)'.
> Also using !IsWindow() instead of !hwnd would fix one more test case.

Thanks, will get into try 2

user32: Partially implement FlashWindowEx with tests

[Detlef Riekenberg]

On So, 2011-05-22, André Hentschel wrote
> Thanks, will get into try 2
> pfwi->cbSize < sizeof(FLASHWINFO) || pfwi->cbSize > sizeof(FLASHWINFO)

That code can be reduced to:

pfwi->cbSize != sizeof(FLASHWINFO)