Thanks, that's what I thought too. I've filed a bug report - see bug 230021


Christian Biesinger wrote:
rblah wrote:

I'm a bit confused about the code in QueryInterface:

    if (aResult != NULL)
    {
        return NS_ERROR_NO_INTERFACE;
    }


Shouldn't aResult contain a reference to an interface? Why isn't this code something like "if (aResult == NULL)" ??


Hm, I would agree with you. I'd say the check should look like this:

if (*aResult == NULL)
    return NS_ERROR_NO_INTERFACE;


_______________________________________________ Mozilla-xpcom mailing list [EMAIL PROTECTED] http://mail.mozilla.org/listinfo/mozilla-xpcom

Reply via email to