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.