Vincent Povirk : mscoree: Always return S_FALSE from DllCanUnloadNow.
Alexandre Julliard
julliard at winehq.org
Thu Feb 24 11:43:25 CST 2011
Module: wine
Branch: master
Commit: b92d834d36fe9f5df42def9e8c8ea589972e34e8
URL: http://source.winehq.org/git/wine.git/?a=commit;h=b92d834d36fe9f5df42def9e8c8ea589972e34e8
Author: Vincent Povirk <vincent at codeweavers.com>
Date: Wed Feb 23 15:29:58 2011 -0600
mscoree: Always return S_FALSE from DllCanUnloadNow.
We can't really unload after Mono is loaded unless we're exiting the process.
We could before that, but it's probably not worth the effort.
---
dlls/mscoree/corruntimehost.c | 7 -------
dlls/mscoree/metadata.c | 4 ----
dlls/mscoree/metahost.c | 8 --------
dlls/mscoree/mscoree_main.c | 7 +------
dlls/mscoree/mscoree_private.h | 4 ----
5 files changed, 1 insertions(+), 29 deletions(-)
diff --git a/dlls/mscoree/corruntimehost.c b/dlls/mscoree/corruntimehost.c
index 0254753..bcdd485 100644
--- a/dlls/mscoree/corruntimehost.c
+++ b/dlls/mscoree/corruntimehost.c
@@ -95,8 +95,6 @@ static HRESULT RuntimeHost_AddDomain(RuntimeHost *This, MonoDomain **result)
list_add_tail(&This->domains, &entry->entry);
- MSCOREE_LockModule();
-
*result = entry->domain;
end:
@@ -137,7 +135,6 @@ static void RuntimeHost_DeleteDomain(RuntimeHost *This, MonoDomain *domain)
if (This->default_domain == domain)
This->default_domain = NULL;
HeapFree(GetProcessHeap(), 0, entry);
- MSCOREE_UnlockModule();
break;
}
}
@@ -183,8 +180,6 @@ static ULONG WINAPI corruntimehost_AddRef(ICorRuntimeHost* iface)
{
RuntimeHost *This = impl_from_ICorRuntimeHost( iface );
- MSCOREE_LockModule();
-
return InterlockedIncrement( &This->ref );
}
@@ -193,8 +188,6 @@ static ULONG WINAPI corruntimehost_Release(ICorRuntimeHost* iface)
RuntimeHost *This = impl_from_ICorRuntimeHost( iface );
ULONG ref;
- MSCOREE_UnlockModule();
-
ref = InterlockedDecrement( &This->ref );
return ref;
diff --git a/dlls/mscoree/metadata.c b/dlls/mscoree/metadata.c
index 7a5336d..a2db508 100644
--- a/dlls/mscoree/metadata.c
+++ b/dlls/mscoree/metadata.c
@@ -79,8 +79,6 @@ static ULONG WINAPI MetaDataDispenser_AddRef(IMetaDataDispenserEx* iface)
TRACE("%p ref=%u\n", This, ref);
- MSCOREE_LockModule();
-
return ref;
}
@@ -96,8 +94,6 @@ static ULONG WINAPI MetaDataDispenser_Release(IMetaDataDispenserEx* iface)
HeapFree(GetProcessHeap(), 0, This);
}
- MSCOREE_UnlockModule();
-
return ref;
}
diff --git a/dlls/mscoree/metahost.c b/dlls/mscoree/metahost.c
index 72215f0..708f638 100644
--- a/dlls/mscoree/metahost.c
+++ b/dlls/mscoree/metahost.c
@@ -325,13 +325,11 @@ static HRESULT WINAPI CLRRuntimeInfo_QueryInterface(ICLRRuntimeInfo* iface,
static ULONG WINAPI CLRRuntimeInfo_AddRef(ICLRRuntimeInfo* iface)
{
- MSCOREE_LockModule();
return 2;
}
static ULONG WINAPI CLRRuntimeInfo_Release(ICLRRuntimeInfo* iface)
{
- MSCOREE_UnlockModule();
return 1;
}
@@ -788,8 +786,6 @@ static ULONG WINAPI InstalledRuntimeEnum_AddRef(IEnumUnknown* iface)
struct InstalledRuntimeEnum *This = (struct InstalledRuntimeEnum*)iface;
ULONG ref = InterlockedIncrement(&This->ref);
- MSCOREE_LockModule();
-
TRACE("(%p) refcount=%u\n", iface, ref);
return ref;
@@ -800,8 +796,6 @@ static ULONG WINAPI InstalledRuntimeEnum_Release(IEnumUnknown* iface)
struct InstalledRuntimeEnum *This = (struct InstalledRuntimeEnum*)iface;
ULONG ref = InterlockedDecrement(&This->ref);
- MSCOREE_UnlockModule();
-
TRACE("(%p) refcount=%u\n", iface, ref);
if (ref == 0)
@@ -942,13 +936,11 @@ static HRESULT WINAPI CLRMetaHost_QueryInterface(ICLRMetaHost* iface,
static ULONG WINAPI CLRMetaHost_AddRef(ICLRMetaHost* iface)
{
- MSCOREE_LockModule();
return 2;
}
static ULONG WINAPI CLRMetaHost_Release(ICLRMetaHost* iface)
{
- MSCOREE_UnlockModule();
return 1;
}
diff --git a/dlls/mscoree/mscoree_main.c b/dlls/mscoree/mscoree_main.c
index 6901f7a..6f53c8f 100644
--- a/dlls/mscoree/mscoree_main.c
+++ b/dlls/mscoree/mscoree_main.c
@@ -47,8 +47,6 @@
WINE_DEFAULT_DEBUG_CHANNEL( mscoree );
-LONG dll_refs = 0;
-
char *WtoA(LPCWSTR wstr)
{
int length;
@@ -459,10 +457,7 @@ HRESULT WINAPI DllUnregisterServer(void)
HRESULT WINAPI DllCanUnloadNow(VOID)
{
- if (dll_refs)
- return S_FALSE;
- else
- return S_OK;
+ return S_FALSE;
}
INT WINAPI ND_RU1( const void *ptr, INT offset )
diff --git a/dlls/mscoree/mscoree_private.h b/dlls/mscoree/mscoree_private.h
index 16ae5d3..aa0ee2e 100644
--- a/dlls/mscoree/mscoree_private.h
+++ b/dlls/mscoree/mscoree_private.h
@@ -20,10 +20,6 @@
#ifndef __MSCOREE_PRIVATE__
#define __MSCOREE_PRIVATE__
-extern LONG dll_refs;
-static inline void MSCOREE_LockModule(void) { InterlockedIncrement(&dll_refs); }
-static inline void MSCOREE_UnlockModule(void) { InterlockedDecrement(&dll_refs); }
-
extern char *WtoA(LPCWSTR wstr);
extern HRESULT CLRMetaHost_CreateInstance(REFIID riid, void **ppobj);
More information about the wine-cvs
mailing list