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

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/comctl32/tests/listview.c:
> - ret = SendMessageA(hList, LVM_SETBKCOLOR, 0, list_background);
> - ok(ret == TRUE, "got 0x%lx, expected 0x%x\n", ret, TRUE);
> -
> - ret = SendMessageA(hList, LVM_SETTEXTBKCOLOR, 0, text_background);
> - ok(ret == TRUE, "got 0x%lx, expected 0x%x\n", ret, TRUE);
> -
> - hdc = GetDC(hwndparent);
> - GetClientRect(hwndparent, &rc);
> -
> - hTheme = pGetWindowTheme(hList);
> - listitem_is_transparent = hTheme && pIsThemeBackgroundPartiallyTransparent(hTheme, LVP_LISTITEM, 0);
> -
> - /* test parameters when the listview is visible */
> - for (i = 0; i < ARRAY_SIZE(params); i++)
> - {
> -
>    winetest_push_context("lParam=0x%Ix", params[i]);

I prefer a newline after winetest_push_context() and before winetest_pop_context().