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]