On 9/30/19 5:38 PM, Jacek Caban wrote:
Hi Gabriel,
On 9/30/19 2:43 PM, Gabriel Ivăncescu wrote:
{ + if (!lstrcmpW(V_BSTR(pDispParams->rgvarg), L"change me")) + { + /* Change the string to check whether it's copied or not */ + lstrcpyW(V_BSTR(pDispParams->rgvarg), L"changed"); + } + else + ok(!lstrcmpW(V_BSTR(pDispParams->rgvarg), L"changed"), + "unexpected second parameter %s.\n", wine_dbgstr_w(V_BSTR(pDispParams->rgvarg))); + } + else + ok(0, "unexpected non-string second parameter V_VT = %d.\n", V_VT(pDispParams->rgvarg));
While I can see why it was interesting for your experimentations, this is not really nice nor something that seems important to have in tests. But please remove this part. The rest of the patch looks good.
Thanks,
Jacek
Ok, I will resend it tomorrow after the others committed so it succeeds. :-)