Re: [PATCH 2/2] vbscript: Fixed CStr

[Jacek Caban]

Hi Shuai,

On 06/13/14 06:26, Shuai Meng wrote:
> -    hres = to_string(arg, &str);
> +    assert(args_cnt == 1);
> +
> +    V_VT(&v) = VT_EMPTY;
> +    hres = VariantChangeType(&v, arg, VARIANT_LOCALBOOL, VT_BSTR);

Why do you change it? to_string() already does the same call.

Jacek

[PATCH 2/2] vbscript: Fixed CStr

[Shuai Meng]

2014-06-13 19:45 GMT+08:00 Jacek Caban <jacek(a)codeweavers.com>:
> Hi Shuai,
>
> On 06/13/14 06:26, Shuai Meng wrote:
> > -    hres = to_string(arg, &str);
> > +    assert(args_cnt == 1);
> > +
> > +    V_VT(&v) = VT_EMPTY;
> > +    hres = VariantChangeType(&v, arg, VARIANT_LOCALBOOL, VT_BSTR);
>
> Why do you change it? to_string() already does the same call.

Yes, you are right, I made a mistake, it's unnecessary  to change it.
> Jacek