Re: [PATCH 3/3] mstask: Implement ClassFactory, DllGetClassObject, and DllCanUnloadNow

[Rob Shearman]

2008/7/18 Roy Shea <royshea(a)gmail.com>:
> +static HRESULT WINAPI MSTASK_IClassFactory_QueryInterface(
> +        LPCLASSFACTORY iface,
> +        REFIID riid,
> +        LPVOID *ppvObj)
> +{
> +    ClassFactoryImpl *This = (ClassFactoryImpl *)iface;
> +
> +    TRACE("\n\tIID:\t%s\n",debugstr_guid(riid));
> +    if (This == NULL || ppvObj == NULL)
> +        return E_POINTER;

Please don't introduce any more checks for This being NULL.

-- 
Rob Shearman