Shuai Meng <mengshuaicalendr@gmail.com> wrote:

> +        switch(V_VT(arg)) {
> +        case VT_UI1:
> +        case VT_I2:
> +        case VT_I4:
> +        case VT_I8:
> +        case VT_R4:
> +        case VT_R8:
> +        case VT_BOOL:
> +        case VT_EMPTY:
> +        case VT_CY:
> +            V_BOOL(res) = VARIANT_TRUE;
> +            break;

This list is far from being complete. It seems that it should also
contain at least VT_UI2, VT_UI4, VT_UI8, VT_I1, VT_INT, VT_UINT.
VT_VECTOR, VT_ARRAY and VT_BYREF modifers probably also should be
taken into account.

--
Dmitry.