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

[Rob Shearman]

2008/7/18 Roy Shea :
> +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.