2008/7/18 Roy Shea <[EMAIL PROTECTED]>:
> +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
