Re: [PATCH v7 0/6] MR6640: propsys: Support converting to BSTR for PropVariantToVariant.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/propsys/tests/propsys.c:
> +    propvar.vt = VT_UI4;
> +    propvar.lVal = 123456;
> +    hr = PropVariantToString(&propvar, bufferW, 4);
> +    todo_wine
> +    ok(hr == STRSAFE_E_INSUFFICIENT_BUFFER, "PropVariantToString returned: %#lx.\n", hr);
> +    todo_wine
> +    ok(!wcscmp(bufferW, L"123"), "Unexpected string %s.\n", debugstr_w(bufferW));
> +    memset(bufferW, 0, sizeof(bufferW));
> +
>      PropVariantInit(&propvar);
>      propvar.vt = VT_LPWSTR;
>      propvar.pwszVal = stringW;
>      hr = PropVariantToString(&propvar, bufferW, 4);
>      ok(hr == STRSAFE_E_INSUFFICIENT_BUFFER, "PropVariantToString returned: 0x%08lx.\n", hr);
> -    ok(!memcmp(bufferW, stringW, 4), "got wrong string.\n");
> +    ok(!wcscmp(bufferW, stringW_truncated), "Unexpected string %s.\n", debugstr_w(bufferW));
> I would probably use a literal string here, seems more readable.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/6640#note_86901