Re: [PATCH 2/3] qmgr: AddRef, QueryInterface, and stub functions for queue manager interface.

[Alexandre Julliard]

Roy Shea <roy(a)cs.hmc.edu> 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
julliard(a)winehq.org