attemp at implementing DllCanUnload now for dcompos
Robert Shearman
rob at codeweavers.com
Wed Dec 1 17:44:55 CST 2004
James Hawkins wrote:
>Hey all,
>
>It would really help me out if you could look over this patch and see
>if I've incorrectly implemented ref-counting and DllCanUnloadNow for
>dcompos. I used suggestions from Rob and a sample implementation
>written by him. The only concern I have is that I'm not sure if all
>of these classes are non-heap based.
>
>
>
>------------------------------------------------------------------------
>
>Index: dlls/dmcompos/chordmap.c
>===================================================================
>RCS file: /home/wine/wine/dlls/dmcompos/chordmap.c,v
>retrieving revision 1.7
>diff -u -r1.7 chordmap.c
>--- dlls/dmcompos/chordmap.c 23 Aug 2004 19:39:57 -0000 1.7
>+++ dlls/dmcompos/chordmap.c 1 Dec 2004 21:15:24 -0000
>@@ -54,18 +54,20 @@
>
> ULONG WINAPI IDirectMusicChordMapImpl_IUnknown_AddRef (LPUNKNOWN iface) {
> ICOM_THIS_MULTI(IDirectMusicChordMapImpl, UnknownVtbl, iface);
>- TRACE("(%p): AddRef from %ld\n", This, This->ref);
>- return ++(This->ref);
>+ TRACE("(%p): AddRef\n", This);
>+
>+ LockModule();
>+
>+ return 2; /* non-heap based object */
> }
>
> ULONG WINAPI IDirectMusicChordMapImpl_IUnknown_Release (LPUNKNOWN iface) {
> ICOM_THIS_MULTI(IDirectMusicChordMapImpl, UnknownVtbl, iface);
>- ULONG ref = --This->ref;
>- TRACE("(%p): ReleaseRef to %ld\n", This, This->ref);
>- if (ref == 0) {
>- HeapFree(GetProcessHeap(), 0, This);
>- }
>- return ref;
>+ TRACE("(%p): ReleaseRef\n", This);
>+
>+ UnlockModule();
>+
>+ return 1; /* non-heap based object */
> }
>
>
No, this isn't a non-heap based object. The code you removed actually
frees the object from the heap and further down in the file the
constructor "DMUSIC_CreateDirectMusicChordMapImpl" allocates it on the
heap. A sure sign of something being heap-based instead of non-heap
based is that it does stuff when its ref count goes to zero, as in the
case above.
Rob
More information about the wine-devel
mailing list