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.