Re: [2/2] mshtml: Add VT_INT support in IHTMLElementCollection_item

[Konstantin Kondratyuk]

Hi, Jacek!

On Friday 19 December 2008 18:04:45 Jacek Caban wrote:
> You're duplicating the code that handles VT_I4 case. Please avoid it.

Do you think, this code will be better? And can I truncate "VT_I4" from trace?

if(V_VT(&name) == VT_I4 || V_VT(&name) == VT_INT) {
        int i;

if (V_VT(&name) == VT_I4) {
            i = V_I4(&name);
            TRACE("name is VT_I4: %d\n", i);
        }
        else {
            i = V_INT(&name);
            TRACE("name is VT_INT: %d\n", i);
        }

if(i < 0)
            return E_INVALIDARG;
        if(i >= This->len)
            return S_OK;

*pdisp = (IDispatch*)This->elems[i];
        IDispatch_AddRef(*pdisp);
        TRACE("Returning pdisp=%p\n", pdisp);
        return S_OK;
    }

And about my test for this problem - it's OK?