Re: [PATCH] vbscript: Implemented Asc

[Nikolay Sivov]

On 02.03.2015 11:12, Shuai Meng wrote:
> - HRESULT hres;
> - BSTR bstr;
> - WCHAR ret;
> -
> - TRACE("(%s)\n", debugstr_variant(arg));
> -
> - switch(V_VT(arg)) {
> - case VT_NULL:
> -
>    return MAKE_VBSERROR(VBSE_ILLEGAL_NULL_USE);
> - case VT_EMPTY:
> -
>    return MAKE_VBSERROR(VBSE_ILLEGAL_FUNC_CALL);
> - case VT_BSTR:
> -
>    bstr = V_BSTR(arg);
> - default:
> -
>    hres = to_string(arg, &bstr);
> -
>    if(FAILED(hres))
> -
>        return hres;
> - }
> -
> - ret = bstr[0];
> - if(ret == 0)
> -
>    return MAKE_VBSERROR(VBSE_ILLEGAL_FUNC_CALL);
> - return return_short(res, ret);

Looks like you need to handle VT_NULL explicitly and after that only 
distinguish between VT_BSTR or the rest. Also you're leaking memory and 
missing a 'break'.