From e3174d93d8c898ea6ecd9c8d8f9bb5b9ff41b9b2 Mon Sep 17 00:00:00 2001 From: Piotr Caban Date: Mon, 30 Jan 2012 17:47:36 +0100 Subject: [PATCH] msvcp60: Fix _Lockit class implementation. --- dlls/msvcp60/misc.c | 59 ++++++-------------------------------------- dlls/msvcp60/msvcp.h | 2 +- 2 files changed, 9 insertions(+), 52 deletions(-) diff --git a/dlls/msvcp60/misc.c b/dlls/msvcp60/misc.c index 94b673bbcb..61cf4a5724 100644 --- a/dlls/msvcp60/misc.c +++ b/dlls/msvcp60/misc.c @@ -89,56 +89,21 @@ void CDECL mutex_mutex_dtor(mutex *m) mutex_dtor(m); } -static CRITICAL_SECTION lockit_cs[_MAX_LOCK]; - -/* ?_Lockit_ctor@_Lockit@std@@SAXH@Z */ -void __cdecl _Lockit_init(int locktype) { - InitializeCriticalSection(&lockit_cs[locktype]); - lockit_cs[locktype].DebugInfo->Spare[0] = (DWORD_PTR)(__FILE__ ": _Lockit critical section"); -} - -/* ?_Lockit_dtor@_Lockit@std@@SAXH@Z */ -void __cdecl _Lockit_free(int locktype) -{ - lockit_cs[locktype].DebugInfo->Spare[0] = 0; - DeleteCriticalSection(&lockit_cs[locktype]); -} +static CRITICAL_SECTION lockit_cs; void init_lockit(void) { - int i; - - for(i=0; i<_MAX_LOCK; i++) - _Lockit_init(i); + InitializeCriticalSection(&lockit_cs); + lockit_cs.DebugInfo->Spare[0] = (DWORD_PTR)(__FILE__ ": _Lockit critical section"); } void free_lockit(void) { - int i; - - for(i=0; i<_MAX_LOCK; i++) - _Lockit_free(i); + lockit_cs.DebugInfo->Spare[0] = 0; + DeleteCriticalSection(&lockit_cs); } -/* ?_Lockit_ctor@_Lockit@std@@CAXPAV12@H@Z */ -/* ?_Lockit_ctor@_Lockit@std@@CAXPEAV12@H@Z */ -void __cdecl _Lockit__Lockit_ctor_locktype(_Lockit *lockit, int locktype) -{ - lockit->locktype = locktype; - EnterCriticalSection(&lockit_cs[locktype]); -} - -/* ?_Lockit_ctor@_Lockit@std@@CAXPAV12@@Z */ -/* ?_Lockit_ctor@_Lockit@std@@CAXPEAV12@@Z */ -void __cdecl _Lockit__Lockit_ctor(_Lockit *lockit) -{ - _Lockit__Lockit_ctor_locktype(lockit, 0); -} - -/* ??0_Lockit@std@@QAE@H@Z */ -/* ??0_Lockit@std@@QEAA@H@Z */ -DEFINE_THISCALL_WRAPPER(_Lockit_ctor_locktype, 8) _Lockit* __thiscall _Lockit_ctor_locktype(_Lockit *this, int locktype) { - _Lockit__Lockit_ctor_locktype(this, locktype); + EnterCriticalSection(&lockit_cs); return this; } @@ -147,15 +112,7 @@ _Lockit* __thiscall _Lockit_ctor_locktype(_Lockit *this, int locktype) DEFINE_THISCALL_WRAPPER(_Lockit_ctor, 4) _Lockit* __thiscall _Lockit_ctor(_Lockit *this) { - _Lockit__Lockit_ctor_locktype(this, 0); - return this; -} - -/* ?_Lockit_dtor@_Lockit@std@@CAXPAV12@@Z */ -/* ?_Lockit_dtor@_Lockit@std@@CAXPEAV12@@Z */ -void __cdecl _Lockit__Lockit_dtor(_Lockit *lockit) -{ - LeaveCriticalSection(&lockit_cs[lockit->locktype]); + return _Lockit_ctor_locktype(this, 0); } /* ??1_Lockit@std@@QAE@XZ */ @@ -163,7 +120,7 @@ void __cdecl _Lockit__Lockit_dtor(_Lockit *lockit) DEFINE_THISCALL_WRAPPER(_Lockit_dtor, 4) void __thiscall _Lockit_dtor(_Lockit *this) { - _Lockit__Lockit_dtor(this); + LeaveCriticalSection(&lockit_cs); } /* wctype */ diff --git a/dlls/msvcp60/msvcp.h b/dlls/msvcp60/msvcp.h index ed31419086..ef6f864044 100644 --- a/dlls/msvcp60/msvcp.h +++ b/dlls/msvcp60/msvcp.h @@ -256,7 +256,7 @@ void __thiscall locale_dtor(locale*); /* class _Lockit */ typedef struct { - int locktype; + char empty_struct; } _Lockit; #define _LOCK_LOCALE 0 -- 2.32.0.93.g670b81a890