[PATCH v6 6/6] vbscript/tests: Add tests for pending and executed code persistence.

Jacek Caban jacek at codeweavers.com
Fri Nov 8 11:28:04 CST 2019


Hi Gabriel,

On 11/8/19 4:48 PM, Gabriel Ivăncescu wrote:
> +    str = a2bstr(
> +        "x = 1\n"
> +        "dim y\ny = 2\n");
> +    hr = IActiveScriptParse_ParseScriptText(parser, str, NULL, NULL, NULL, 0, 0, 0, NULL, NULL);
> +    ok(hr == S_OK, "ParseScriptText failed: %08x\n", hr);
> +    SysFreeString(str);


Please avoid using a2bstr. The plan is to remove it now that we can use 
wchar literals. In this case ParseScriptText does not even take BSTR as 
an argument, so you may just pass L"" string directly.


Thanks,

Jacek




More information about the wine-devel mailing list