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

Alexandre Julliard julliard at winehq.org
Wed Dec 19 06:20:16 CST 2007


Roy Shea <roy at 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 at winehq.org



More information about the wine-devel mailing list