Re: [PATCH v2 0/1] MR2367: comctl32/treeview: Ignore the lParam to WM_PRINTCLIENT and add tests.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/comctl32/tests/treeview.c:
> +{
> +    static const LPARAM params[] = {0, PRF_CHECKVISIBLE, PRF_NONCLIENT, PRF_CLIENT, PRF_ERASEBKGND,
> +                                    PRF_CHILDREN, PRF_OWNED};
> +    BOOL glyph_is_transparent;
> +    HTHEME hTheme;
> +    HWND hTree;
> +    COLORREF clr;
> +    LONG ret;
> +    RECT rc;
> +    HDC hdc;
> +    int i;
> +
> +    hTree = create_treeview_control(0);
> +
> +    clr = SendMessageA(hTree, TVM_SETBKCOLOR, 0, RGB(255, 0, 0));
> +    ok(clr == ~0u, "got %ld, expected -1\n", clr);
> You can use CLR_NONE instead of ~0u. 'expected -1' is wrong.

Otherwise, this patch looks good. Thanks for looking into it.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2367#note_26840