Re: [PATCH 2/5] user32: Add initial out of window procedure slots test

[Dmitry Timoshkov]

Piotr Caban <piotr(a)codeweavers.com> wrote:
> +static LRESULT WINAPI winproc(HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam)
> +{
> +    if(!hwnd) {
> +        int *count = (int*)lparam;
> +        (*count)++;
> +    }
> +    return 0;
> +}
> +
> +static LRESULT WINAPI winproc2(HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam)
> +{
> +    return 0;
> +}
> ...
> +    memset(&wnd_classA, 0, sizeof(wnd_classA));
> +    wnd_classA.lpszClassName = "winproc_test";
> +    wnd_classA.lpfnWndProc = &winproc;

Should be just "winproc" instead of "&winproc"
> +    ok(SetWindowLongPtrA(hwnd, GWLP_WNDPROC, (LONG_PTR)&winproc2),
> +            "SetWindowLongPtr failed with error %d\n", GetLastError());

Same here.

-- 
Dmitry.

[PATCH 2/5] user32: Add initial out of window procedure slots test

[Piotr Caban]

On 03/09/16 01:57, Dmitry Timoshkov wrote:
> Piotr Caban <piotr(a)codeweavers.com> wrote:
> > +    memset(&wnd_classA, 0, sizeof(wnd_classA));
> > +    wnd_classA.lpszClassName = "winproc_test";
> > +    wnd_classA.lpfnWndProc = &winproc;
>
> Should be just "winproc" instead of "&winproc"

I've sent a fixed version.

Thanks,
Piotr