Re: [PATCH 1/3] mmdevapi: Add class factory

[Alexandre Julliard]

Maarten Lankhorst  writes:
> +static ULONG WINAPI MMCF_Release(LPCLASSFACTORY iface)
> +{
> - IClassFactoryImpl *This = (IClassFactoryImpl *)iface;
> - ULONG ref = InterlockedDecrement(&(This->ref));
> - TRACE("(%p) ref was %d\n", This, ref + 1);
> - /* static class, won't be freed */

You don't need a refcount then.

[PATCH 1/3] mmdevapi: Add class factory

[Maarten Lankhorst]

Hi Alexandre,

2009/12/15 Alexandre Julliard :
> Maarten Lankhorst  writes:
> > +static ULONG WINAPI MMCF_Release(LPCLASSFACTORY iface)
> > +{
> > - IClassFactoryImpl *This = (IClassFactoryImpl *)iface;
> > - ULONG ref = InterlockedDecrement(&(This->ref));
> > - TRACE("(%p) ref was %d\n", This, ref + 1);
> > - /* static class, won't be freed */
>
> You don't need a refcount then.

I just copied the dsound class factory code, if you want I can send
patches for both, figured that copying existing code was the best way.
But I suppose I' ll send a patch for both then.

Cheers,
Maarten.