Re: [PATCH v5 0/2] MR10340: comctl32/tooltips: suppress icon and title when title is NULL or empty.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/comctl32/tests/tooltips.c:
> +    SendMessageW(tt, TTM_SETTITLEW, TTI_WARNING, (LPARAM)NULL);
> +    get_tracked_size(tt, &info, &sz_null_icon);
> +    todo_wine ok(sz_null_icon.cy == sz_no_title.cy,
> +       "NULL title + icon: height %ld, expected %ld (same as no-title)\n",
> +       sz_null_icon.cy, sz_no_title.cy);
> +    todo_wine ok(sz_null_icon.cx == sz_no_title.cx,
> +       "NULL title + icon: width %ld, expected %ld (same as no-title)\n",
> +       sz_null_icon.cx, sz_no_title.cx);
> +
> +    /* Icon requested with empty string title — must also be suppressed */
> +    SendMessageW(tt, TTM_SETTITLEW, TTI_WARNING, (LPARAM)L"");
> +    get_tracked_size(tt, &info, &sz_empty_icon);
> +    todo_wine ok(sz_empty_icon.cy == sz_no_title.cy,
> +       "empty title + icon: height %ld, expected %ld (same as no-title)\n",
> +       sz_empty_icon.cy, sz_no_title.cy);
> +    todo_wine ok(sz_empty_icon.cx == sz_no_title.cx,
> Let's put todo_wine on a newline. For example,
    todo_wine
    ok(...

And let's check width and height in one ok() call.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10340#note_137912