Roy Shea <[EMAIL PROTECTED]> writes:

> +static ULONG WINAPI BITS_IBackgroundCopyManager_AddRef(
> +        IBackgroundCopyManager* iface)
> +{
> +    ICOM_THIS_MULTI(BackgroundCopyManagerImpl, lpVtbl, iface);

There's no reason to use ICOM_THIS_MULTI with a single vtbl.

> +    if (This == NULL)
> +    {
> +        return E_POINTER;
> +    }

Checking This for NULL is a waste of time.

-- 
Alexandre Julliard
[EMAIL PROTECTED]


Reply via email to