October 2009 - Wine-commits

Alexandre Julliard : user32: Fix the return value of EnumThreadWindows.
by Alexandre Julliard 23 Oct '09

23 Oct '09

Module: wine Branch: master Commit: 1b27890af69f78f657ddac886b9eaecf4372cf28 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1b27890af69f78f657ddac886… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Oct 23 14:46:58 2009 +0200 user32: Fix the return value of EnumThreadWindows. --- dlls/user32/tests/win.c | 55 +++++++++++++++++++++++++++++++++++++++++++++++ dlls/user32/win.c | 5 ++- 2 files changed, 58 insertions(+), 2 deletions(-) diff --git a/dlls/user32/tests/win.c b/dlls/user32/tests/win.c index 63f25bc..1c9103a 100644 --- a/dlls/user32/tests/win.c +++ b/dlls/user32/tests/win.c @@ -569,6 +569,60 @@ static void test_parent_owner(void) DestroyWindow( owner ); } +static BOOL CALLBACK enum_proc( HWND hwnd, LPARAM lParam) +{ + (*(LPINT)lParam)++; + if (*(LPINT)lParam > 2) return FALSE; + return TRUE; +} +static DWORD CALLBACK enum_thread( void *arg ) +{ + INT count; + HWND hwnd[3]; + BOOL ret; + MSG msg; + + PeekMessage( &msg, 0, 0, 0, PM_NOREMOVE ); /* make sure we have a message queue */ + + count = 0; + ret = EnumThreadWindows( GetCurrentThreadId(), enum_proc, (LPARAM)&count ); + ok( ret, "EnumThreadWindows should have returned TRUE\n" ); + ok( count == 0, "count should be 0 got %d\n", count ); + + hwnd[0] = CreateWindowExA(0, "ToolWindowClass", "Tool window 1", WS_POPUP, + 0, 0, 100, 100, 0, 0, 0, NULL ); + count = 0; + ret = EnumThreadWindows( GetCurrentThreadId(), enum_proc, (LPARAM)&count ); + ok( ret, "EnumThreadWindows should have returned TRUE\n" ); + if (count != 2) /* Vista gives us two windows for the price of one */ + { + ok( count == 1, "count should be 1 got %d\n", count ); + hwnd[2] = CreateWindowExA(0, "ToolWindowClass", "Tool window 2", WS_POPUP, + 0, 0, 100, 100, 0, 0, 0, NULL ); + } + else hwnd[2] = 0; + + hwnd[1] = CreateWindowExA(0, "ToolWindowClass", "Tool window 3", WS_POPUP, + 0, 0, 100, 100, 0, 0, 0, NULL ); + count = 0; + ret = EnumThreadWindows( GetCurrentThreadId(), enum_proc, (LPARAM)&count ); + ok( !ret, "EnumThreadWindows should have returned FALSE\n" ); + ok( count == 3, "count should be 1 got %d\n", count ); + + if (hwnd[2]) DestroyWindow(hwnd[2]); + DestroyWindow(hwnd[1]); + DestroyWindow(hwnd[0]); + return 0; +} + +/* test EnumThreadWindows in a separate thread */ +static void test_enum_thread_windows(void) +{ + DWORD id; + HANDLE handle = CreateThread( NULL, 0, enum_thread, 0, 0, &id ); + ok( !WaitForSingleObject( handle, 10000 ), "wait failed\n" ); + CloseHandle( handle ); +} static LRESULT WINAPI main_window_procA(HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam) { @@ -5762,6 +5816,7 @@ START_TEST(win) test_CreateWindow(); test_parent_owner(); test_SetParent(); + test_enum_thread_windows(); test_mdi(); test_icons(); diff --git a/dlls/user32/win.c b/dlls/user32/win.c index 3e6dc37..d6746b0 100644 --- a/dlls/user32/win.c +++ b/dlls/user32/win.c @@ -3151,6 +3151,7 @@ BOOL WINAPI EnumThreadWindows( DWORD id, WNDENUMPROC func, LPARAM lParam ) { HWND *list; int i; + BOOL ret = TRUE; USER_CheckNotLock(); @@ -3159,9 +3160,9 @@ BOOL WINAPI EnumThreadWindows( DWORD id, WNDENUMPROC func, LPARAM lParam ) /* Now call the callback function for every window */ for (i = 0; list[i]; i++) - if (!func( list[i], lParam )) break; + if (!(ret = func( list[i], lParam ))) break; HeapFree( GetProcessHeap(), 0, list ); - return TRUE; + return ret; }

1 0

Detlef Riekenberg : localspl: Accept environment 'Windows IA64'.
by Alexandre Julliard 23 Oct '09

23 Oct '09

Module: wine Branch: master Commit: 836ca78fd59af8e710298defcb5744894e67aba0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=836ca78fd59af8e710298defc… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Fri Oct 23 14:01:51 2009 +0200 localspl: Accept environment 'Windows IA64'. --- dlls/localspl/provider.c | 7 ++++++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/dlls/localspl/provider.c b/dlls/localspl/provider.c index 8c6a940..b17e074 100644 --- a/dlls/localspl/provider.c +++ b/dlls/localspl/provider.c @@ -118,6 +118,8 @@ static const WCHAR fmt_printprocessorsW[] = { 'S','y','s','t','e','m','\\', 'P','r','i','n','t',' ','P','r','o','c','e','s','s','o','r','s',0 }; static const WCHAR hardwareidW[] = {'H','a','r','d','w','a','r','e','I','D',0}; static const WCHAR help_fileW[] = {'H','e','l','p',' ','F','i','l','e',0}; +static const WCHAR ia64_envnameW[] = {'W','i','n','d','o','w','s',' ','I','A','6','4',0}; +static const WCHAR ia64_subdirW[] = {'i','a','6','4',0}; static const WCHAR localportW[] = {'L','o','c','a','l',' ','P','o','r','t',0}; static const WCHAR locationW[] = {'L','o','c','a','t','i','o','n',0}; static const WCHAR manufacturerW[] = {'M','a','n','u','f','a','c','t','u','r','e','r',0}; @@ -161,6 +163,9 @@ static const WCHAR XcvMonitorW[] = {',','X','c','v','M','o','n','i','t','o','r', static const WCHAR XcvPortW[] = {',','X','c','v','P','o','r','t',' ',0}; +static const printenv_t env_ia64 = {ia64_envnameW, ia64_subdirW, 3, + version3_regpathW, version3_subdirW}; + static const printenv_t env_x86 = {x86_envnameW, x86_subdirW, 3, version3_regpathW, version3_subdirW}; @@ -170,7 +175,7 @@ static const printenv_t env_x64 = {x64_envnameW, x64_subdirW, 3, static const printenv_t env_win40 = {win40_envnameW, win40_subdirW, 0, version0_regpathW, version0_subdirW}; -static const printenv_t * const all_printenv[] = {&env_x86, &env_x64, &env_win40}; +static const printenv_t * const all_printenv[] = {&env_x86, &env_x64, &env_ia64, &env_win40}; static const DWORD di_sizeof[] = {0, sizeof(DRIVER_INFO_1W), sizeof(DRIVER_INFO_2W),

1 0

Detlef Riekenberg : user32: Bump version to XP values.
by Alexandre Julliard 23 Oct '09

23 Oct '09

Module: wine Branch: master Commit: 6f6ce4a3250216f77897e6fc2c905c50ca8493c8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6f6ce4a3250216f77897e6fc2… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Fri Oct 23 15:02:47 2009 +0200 user32: Bump version to XP values. --- dlls/user32/resources/version.rc | 7 ++++++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/dlls/user32/resources/version.rc b/dlls/user32/resources/version.rc index 228da33..9848a31 100644 --- a/dlls/user32/resources/version.rc +++ b/dlls/user32/resources/version.rc @@ -20,8 +20,13 @@ LANGUAGE LANG_NEUTRAL, SUBLANG_NEUTRAL -#define WINE_FILEVERSION_STR "1.0" #define WINE_FILEDESCRIPTION_STR "Wine core dll" #define WINE_FILENAME_STR "user32.dll" +/* these values come from Windows XP SP2 */ +#define WINE_FILEVERSION 5,1,2600,2180 +#define WINE_FILEVERSION_STR "5.01.2600.2180" +#define WINE_PRODUCTVERSION 5,1,2600,2180 +#define WINE_PRODUCTVERSION_STR "5.01.2600.2180" + #include "wine/wine_common_ver.rc"

1 0

Paul Vriens : user32/tests: Skip some tests on Win9x/WinMe.
by Alexandre Julliard 23 Oct '09

23 Oct '09

Module: wine Branch: master Commit: 05765a0b0c0f0b3caa8cb4ad10a2a206ddf7f913 URL: http://source.winehq.org/git/wine.git/?a=commit;h=05765a0b0c0f0b3caa8cb4ad1… Author: Paul Vriens <Paul.Vriens.Wine(a)gmail.com> Date: Fri Oct 23 15:03:12 2009 +0200 user32/tests: Skip some tests on Win9x/WinMe. --- dlls/user32/tests/input.c | 7 ++++++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/dlls/user32/tests/input.c b/dlls/user32/tests/input.c index 42d9544..d82ef3c 100644 --- a/dlls/user32/tests/input.c +++ b/dlls/user32/tests/input.c @@ -904,7 +904,12 @@ static void test_Input_blackbox(void) SetWindowPos( window, HWND_TOPMOST, 0, 0, 0, 0, SWP_NOSIZE|SWP_NOMOVE ); SetForegroundWindow( window ); - hook = SetWindowsHookExA(WH_KEYBOARD_LL, hook_proc, GetModuleHandleA( NULL ), 0); + if (!(hook = SetWindowsHookExA(WH_KEYBOARD_LL, hook_proc, GetModuleHandleA( NULL ), 0))) + { + DestroyWindow(window); + win_skip("WH_KEYBOARD_LL is not supported\n"); + return; + } /* must process all initial messages, otherwise X11DRV_KeymapNotify unsets * key state set by SendInput(). */

1 0

Eric Pouech : winmm: Restrict exposure of WINE_MMTHREAD to mmsystem.
by Alexandre Julliard 23 Oct '09

23 Oct '09

Module: wine Branch: master Commit: 73c7919c87a5b2206547211e40d84f478210187b URL: http://source.winehq.org/git/wine.git/?a=commit;h=73c7919c87a5b2206547211e4… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Thu Oct 22 22:10:47 2009 +0200 winmm: Restrict exposure of WINE_MMTHREAD to mmsystem. --- dlls/winmm/driver.c | 6 +++++- dlls/winmm/mmsystem.c | 2 -- dlls/winmm/winemm.h | 19 ------------------- dlls/winmm/winemm16.h | 18 ++++++++++++++++++ 4 files changed, 23 insertions(+), 22 deletions(-) diff --git a/dlls/winmm/driver.c b/dlls/winmm/driver.c index 317da60..58afcba 100644 --- a/dlls/winmm/driver.c +++ b/dlls/winmm/driver.c @@ -53,7 +53,6 @@ static LPWINE_DRIVER lpDrvItemList /* = NULL */; static const WCHAR HKLM_BASE[] = {'S','o','f','t','w','a','r','e','\\','M','i','c','r','o','s','o','f','t','\\', 'W','i','n','d','o','w','s',' ','N','T','\\','C','u','r','r','e','n','t','V','e','r','s','i','o','n',0}; -WINE_MMTHREAD* (*pFnGetMMThread16)(UINT16 h) /* = NULL */; LPWINE_DRIVER (*pFnOpenDriver16)(LPCWSTR,LPCWSTR,LPARAM) /* = NULL */; LRESULT (*pFnCloseDriver16)(UINT16,LPARAM,LPARAM) /* = NULL */; LRESULT (*pFnSendMessage16)(UINT16,UINT,LPARAM,LPARAM) /* = NULL */; @@ -598,6 +597,10 @@ BOOL WINAPI DriverCallback(DWORD_PTR dwCallBack, DWORD uFlags, HDRVR hDev, TRACE("Event(%08lx) !\n", dwCallBack); SetEvent((HANDLE)dwCallBack); break; +#if 0 + /* FIXME: for now only usable in mmsystem.dll16 + * If needed, should be enabled back + */ case 6: /* I would dub it DCB_MMTHREADSIGNAL */ /* this is an undocumented DCB_ value used for mmThreads * loword of dwCallBack contains the handle of the lpMMThd block @@ -614,6 +617,7 @@ BOOL WINAPI DriverCallback(DWORD_PTR dwCallBack, DWORD uFlags, HDRVR hDev, /* some other stuff on lpMMThd->hVxD */ } break; +#endif #if 0 case 4: /* this is an undocumented DCB_ value for... I don't know */ diff --git a/dlls/winmm/mmsystem.c b/dlls/winmm/mmsystem.c index bf66e45..daf78d0 100644 --- a/dlls/winmm/mmsystem.c +++ b/dlls/winmm/mmsystem.c @@ -87,7 +87,6 @@ BOOL WINAPI MMSYSTEM_LibMain(DWORD fdwReason, HINSTANCE hinstDLL, WORD ds, return FALSE; } /* hook in our 16 bit function pointers */ - pFnGetMMThread16 = WINMM_GetmmThread; pFnOpenDriver16 = DRIVER_OpenDriver16; pFnCloseDriver16 = DRIVER_CloseDriver16; pFnSendMessage16 = DRIVER_SendMessage16; @@ -96,7 +95,6 @@ BOOL WINAPI MMSYSTEM_LibMain(DWORD fdwReason, HINSTANCE hinstDLL, WORD ds, MMDRV_Init16(); break; case DLL_PROCESS_DETACH: - pFnGetMMThread16 = NULL; pFnOpenDriver16 = NULL; pFnCloseDriver16 = NULL; pFnSendMessage16 = NULL; diff --git a/dlls/winmm/winemm.h b/dlls/winmm/winemm.h index 8bf2cae..a06daf2 100644 --- a/dlls/winmm/winemm.h +++ b/dlls/winmm/winemm.h @@ -115,24 +115,6 @@ typedef struct { WINE_MLD mld; } WINE_MIXER, *LPWINE_MIXER; -#define WINE_MMTHREAD_CREATED 0x4153494C /* "BSIL" */ -#define WINE_MMTHREAD_DELETED 0xDEADDEAD - -typedef struct { - DWORD dwSignature; /* 00 "BSIL" when ok, 0xDEADDEAD when being deleted */ - DWORD dwCounter; /* 04 > 1 when in mmThread functions */ - HANDLE hThread; /* 08 hThread */ - DWORD dwThreadID; /* 0C */ - DWORD fpThread; /* 10 address of thread proc (segptr or lin depending on dwFlags) */ - DWORD dwThreadPmt; /* 14 parameter to be passed upon thread creation to fpThread */ - LONG dwSignalCount; /* 18 counter used for signaling */ - HANDLE hEvent; /* 1C event */ - HANDLE hVxD; /* 20 return from OpenVxDHandle */ - DWORD dwStatus; /* 24 0x00, 0x10, 0x20, 0x30 */ - DWORD dwFlags; /* 28 dwFlags upon creation */ - UINT16 hTask; /* 2C handle to created task */ -} WINE_MMTHREAD; - typedef struct tagWINE_MCIDRIVER { UINT wDeviceID; UINT wType; @@ -212,7 +194,6 @@ extern HANDLE psStopEvent; /* pointers to 16 bit functions (if sibling MMSYSTEM.DLL is loaded * NULL otherwise */ -extern WINE_MMTHREAD* (*pFnGetMMThread16)(UINT16); extern LPWINE_DRIVER (*pFnOpenDriver16)(LPCWSTR,LPCWSTR,LPARAM); extern LRESULT (*pFnCloseDriver16)(UINT16,LPARAM,LPARAM); extern LRESULT (*pFnSendMessage16)(UINT16,UINT,LPARAM,LPARAM); diff --git a/dlls/winmm/winemm16.h b/dlls/winmm/winemm16.h index 97b03ef..2435e90 100644 --- a/dlls/winmm/winemm16.h +++ b/dlls/winmm/winemm16.h @@ -79,3 +79,21 @@ extern void MMSYSTDRV_DeleteThunk(struct mmsystdrv_thunk* th extern void MMSYSTDRV_SetHandle(struct mmsystdrv_thunk* thunk, void* h); extern void MMSYSTDRV_CloseHandle(void* h); extern DWORD MMSYSTDRV_Message(void* h, UINT msg, DWORD_PTR param1, DWORD_PTR param2); + +#define WINE_MMTHREAD_CREATED 0x4153494C /* "BSIL" */ +#define WINE_MMTHREAD_DELETED 0xDEADDEAD + +typedef struct { + DWORD dwSignature; /* 00 "BSIL" when ok, 0xDEADDEAD when being deleted */ + DWORD dwCounter; /* 04 > 1 when in mmThread functions */ + HANDLE hThread; /* 08 hThread */ + DWORD dwThreadID; /* 0C */ + DWORD fpThread; /* 10 address of thread proc (segptr or lin depending on dwFlags) */ + DWORD dwThreadPmt; /* 14 parameter to be passed upon thread creation to fpThread */ + LONG dwSignalCount; /* 18 counter used for signaling */ + HANDLE hEvent; /* 1C event */ + HANDLE hVxD; /* 20 return from OpenVxDHandle */ + DWORD dwStatus; /* 24 0x00, 0x10, 0x20, 0x30 */ + DWORD dwFlags; /* 28 dwFlags upon creation */ + UINT16 hTask; /* 2C handle to created task */ +} WINE_MMTHREAD;

1 0

Eric Pouech : winmm: Duplicate the 16<> 32 time struct conversion functions in both DLLs.
by Alexandre Julliard 23 Oct '09

23 Oct '09

Module: wine Branch: master Commit: 9709d1c67f09c5f9541b0cc12a8a31731a081799 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9709d1c67f09c5f9541b0cc12… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Thu Oct 22 22:10:54 2009 +0200 winmm: Duplicate the 16<>32 time struct conversion functions in both DLLs. --- dlls/winmm/message16.c | 17 +++++++++++++++-- 1 files changed, 15 insertions(+), 2 deletions(-) diff --git a/dlls/winmm/message16.c b/dlls/winmm/message16.c index d13190e..0193683 100644 --- a/dlls/winmm/message16.c +++ b/dlls/winmm/message16.c @@ -33,6 +33,19 @@ WINE_DEFAULT_DEBUG_CHANNEL(winmm); +/****************************************************************** + * WINMM_MMTIME16to32 + */ +static void WINMM_MMTIME16to32(LPMMTIME mmt32, const MMTIME16* mmt16) +{ + mmt32->wType = mmt16->wType; + /* layout of rest is the same for 32/16, + * Note: mmt16->u is 2 bytes smaller than mmt32->u, which has padding + */ + memcpy(&(mmt32->u), &(mmt16->u), sizeof(mmt16->u)); +} + + /************************************************************************** * MMDRV_Callback [internal] */ @@ -1165,7 +1178,7 @@ static WINMM_MapType MMDRV_WaveIn_UnMap32WTo16(UINT wMsg, DWORD_PTR *lpdwUser, LPSTR ptr = (LPSTR)mmt16 - sizeof(LPMMTIME); LPMMTIME mmt32 = *(LPMMTIME*)ptr; - MMSYSTEM_MMTIME16to32(mmt32, mmt16); + WINMM_MMTIME16to32(mmt32, mmt16); UnMapLS( *lpParam1 ); HeapFree( GetProcessHeap(), 0, ptr ); ret = WINMM_MAP_OK; @@ -1689,7 +1702,7 @@ static WINMM_MapType MMDRV_WaveOut_UnMap32WTo16(UINT wMsg, DWORD_PTR *lpdwUser, LPSTR ptr = (LPSTR)mmt16 - sizeof(LPMMTIME); LPMMTIME mmt32 = *(LPMMTIME*)ptr; - MMSYSTEM_MMTIME16to32(mmt32, mmt16); + WINMM_MMTIME16to32(mmt32, mmt16); UnMapLS( *lpParam1 ); HeapFree( GetProcessHeap(), 0, ptr ); ret = WINMM_MAP_OK;

1 0

Eric Pouech : winmm: Simply use kernel32. {Release|Restore}ThunkLock instead of using 16bit convolutions.
by Alexandre Julliard 23 Oct '09

23 Oct '09

Module: wine Branch: master Commit: 14a68405d3295811fedce0c1da4fdb6e7d7a6936 URL: http://source.winehq.org/git/wine.git/?a=commit;h=14a68405d3295811fedce0c1d… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Thu Oct 22 22:11:01 2009 +0200 winmm: Simply use kernel32.{Release|Restore}ThunkLock instead of using 16bit convolutions. --- dlls/winmm/mmsystem.c | 4 ---- dlls/winmm/winemm.h | 2 -- dlls/winmm/winmm.c | 11 ++++------- 3 files changed, 4 insertions(+), 13 deletions(-) diff --git a/dlls/winmm/mmsystem.c b/dlls/winmm/mmsystem.c index daf78d0..ce26192 100644 --- a/dlls/winmm/mmsystem.c +++ b/dlls/winmm/mmsystem.c @@ -90,16 +90,12 @@ BOOL WINAPI MMSYSTEM_LibMain(DWORD fdwReason, HINSTANCE hinstDLL, WORD ds, pFnOpenDriver16 = DRIVER_OpenDriver16; pFnCloseDriver16 = DRIVER_CloseDriver16; pFnSendMessage16 = DRIVER_SendMessage16; - pFnReleaseThunkLock = ReleaseThunkLock; - pFnRestoreThunkLock = RestoreThunkLock; MMDRV_Init16(); break; case DLL_PROCESS_DETACH: pFnOpenDriver16 = NULL; pFnCloseDriver16 = NULL; pFnSendMessage16 = NULL; - pFnReleaseThunkLock = NULL; - pFnRestoreThunkLock = NULL; /* FIXME: add equivalent for MMDRV_Init16() */ break; case DLL_THREAD_ATTACH: diff --git a/dlls/winmm/winemm.h b/dlls/winmm/winemm.h index a06daf2..4eabe70 100644 --- a/dlls/winmm/winemm.h +++ b/dlls/winmm/winemm.h @@ -201,8 +201,6 @@ extern WINMM_MapType (*pFnMciMapMsg32WTo16)(WORD,WORD,DWORD,DWORD_PTR*); extern WINMM_MapType (*pFnMciUnMapMsg32WTo16)(WORD,WORD,DWORD,DWORD_PTR); extern LRESULT (*pFnCallMMDrvFunc16)(DWORD /* in fact FARPROC16 */,WORD,WORD,LONG,LONG,LONG); extern unsigned (*pFnLoadMMDrvFunc16)(LPCSTR,LPWINE_DRIVER, LPWINE_MM_DRIVER); -extern void (WINAPI *pFnReleaseThunkLock)(DWORD*); -extern void (WINAPI *pFnRestoreThunkLock)(DWORD); /* GetDriverFlags() returned bits is not documented (nor the call itself) * Here are Wine only definitions of the bits diff --git a/dlls/winmm/winmm.c b/dlls/winmm/winmm.c index 7855bc3..677bfa3 100644 --- a/dlls/winmm/winmm.c +++ b/dlls/winmm/winmm.c @@ -57,9 +57,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(winmm); -void (WINAPI *pFnReleaseThunkLock)(DWORD*); -void (WINAPI *pFnRestoreThunkLock)(DWORD); - /* ======================================================================== * G L O B A L S E T T I N G S * ========================================================================*/ @@ -1748,9 +1745,9 @@ static BOOL MMSYSTEM_MidiStream_PostMessage(WINE_MIDIStream* lpMidiStrm, WORD ms if (PostThreadMessageA(lpMidiStrm->dwThreadID, msg, pmt1, pmt2)) { DWORD count; - if (pFnReleaseThunkLock) pFnReleaseThunkLock(&count); + ReleaseThunkLock(&count); WaitForSingleObject(lpMidiStrm->hEvent, INFINITE); - if (pFnRestoreThunkLock) pFnRestoreThunkLock(count); + RestoreThunkLock(count); } else { WARN("bad PostThreadMessageA\n"); return FALSE; @@ -1836,9 +1833,9 @@ MMRESULT WINAPI midiStreamOpen(HMIDISTRM* lphMidiStrm, LPUINT lpuDeviceID, * (meaning the Win16Lock is set), so that it's released and the 32 bit thread running * MMSYSTEM_MidiStreamPlayer can acquire Win16Lock to create its queue. */ - if (pFnReleaseThunkLock) pFnReleaseThunkLock(&count); + ReleaseThunkLock(&count); WaitForSingleObject(lpMidiStrm->hEvent, INFINITE); - if (pFnRestoreThunkLock) pFnRestoreThunkLock(count); + RestoreThunkLock(count); } TRACE("=> (%u/%d) hMidi=%p ret=%d lpMidiStrm=%p\n",

1 0

Eric Pouech : winmm: Get rid of 16bit driver support in mci.
by Alexandre Julliard 23 Oct '09

23 Oct '09

Module: wine Branch: master Commit: 2a581444d0b92f1a5bf2767f659d28cbc6b88e87 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2a581444d0b92f1a5bf2767f6… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Thu Oct 22 22:11:07 2009 +0200 winmm: Get rid of 16bit driver support in mci. --- dlls/winmm/mci.c | 51 +---- dlls/winmm/message16.c | 685 ------------------------------------------------ dlls/winmm/winemm.h | 3 - 3 files changed, 3 insertions(+), 736 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=2a581444d0b92f1a5bf27…

1 0

Eric Pouech : winmm: Get rid of 16bit driver support.
by Alexandre Julliard 23 Oct '09

23 Oct '09

Module: wine Branch: master Commit: ca1a7c6badb8cd8eea67cbeedde0ebae48fda5e9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ca1a7c6badb8cd8eea67cbeed… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Thu Oct 22 22:11:15 2009 +0200 winmm: Get rid of 16bit driver support. --- dlls/winmm/driver.c | 135 ++---- dlls/winmm/lolvldrv.c | 143 +----- dlls/winmm/message16.c | 1168 ------------------------------------------------ dlls/winmm/mmsystem.c | 258 ----------- dlls/winmm/winemm.h | 47 +-- 5 files changed, 79 insertions(+), 1672 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=ca1a7c6badb8cd8eea67c…

1 0

Eric Pouech : winmm: Final split of mmsystem.
by Alexandre Julliard 23 Oct '09

23 Oct '09

Module: wine Branch: master Commit: a60dd6e9181abc6d94fce490152c0492597178c4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a60dd6e9181abc6d94fce4901… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Thu Oct 22 22:11:29 2009 +0200 winmm: Final split of mmsystem. --- .gitignore | 1 - configure | 9 +++++++++ configure.ac | 1 + dlls/Makefile.in | 4 ---- dlls/mmsystem.dll16/Makefile.in | 18 ++++++++++++++++++ dlls/{winmm => mmsystem.dll16}/mci16.c | 0 dlls/{winmm => mmsystem.dll16}/message16.c | 0 dlls/{winmm => mmsystem.dll16}/mmio16.c | 0 dlls/{winmm => mmsystem.dll16}/mmsystem.c | 12 ++++++------ .../mmsystem.dll16.spec} | 0 dlls/{winmm => mmsystem.dll16}/winemm16.h | 0 dlls/winmm/Makefile.in | 9 --------- 12 files changed, 34 insertions(+), 20 deletions(-) diff --git a/.gitignore b/.gitignore index 5d498c8..2ac3a69 100644 --- a/.gitignore +++ b/.gitignore @@ -64,7 +64,6 @@ dlls/krnl386.exe16 dlls/libd3dx9.def dlls/libwinspool.def dlls/libxinput.def -dlls/mmsystem.dll16 dlls/msdaps/msdaps.h dlls/msdaps/msdaps_i.c dlls/msdaps/msdaps_p.c diff --git a/configure b/configure index 1507ead..32e37b2 100755 --- a/configure +++ b/configure @@ -15351,6 +15351,14 @@ dlls/mmdevldr.vxd/Makefile: dlls/mmdevldr.vxd/Makefile.in dlls/Makedll.rules" ac_config_files="$ac_config_files dlls/mmdevldr.vxd/Makefile" ALL_MAKEFILES="$ALL_MAKEFILES \\ + dlls/mmsystem.dll16/Makefile" +test "x$enable_win16" != xno && ALL_DLL_DIRS="$ALL_DLL_DIRS \\ + mmsystem.dll16" +ALL_MAKEFILE_DEPENDS="$ALL_MAKEFILE_DEPENDS +dlls/mmsystem.dll16/Makefile: dlls/mmsystem.dll16/Makefile.in dlls/Makedll.rules" +ac_config_files="$ac_config_files dlls/mmsystem.dll16/Makefile" + +ALL_MAKEFILES="$ALL_MAKEFILES \\ dlls/monodebg.vxd/Makefile" test "x$enable_win16" != xno && ALL_DLL_DIRS="$ALL_DLL_DIRS \\ monodebg.vxd" @@ -18937,6 +18945,7 @@ do "dlls/mlang/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/mlang/Makefile" ;; "dlls/mlang/tests/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/mlang/tests/Makefile" ;; "dlls/mmdevldr.vxd/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/mmdevldr.vxd/Makefile" ;; + "dlls/mmsystem.dll16/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/mmsystem.dll16/Makefile" ;; "dlls/monodebg.vxd/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/monodebg.vxd/Makefile" ;; "dlls/mountmgr.sys/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/mountmgr.sys/Makefile" ;; "dlls/mouse.drv16/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/mouse.drv16/Makefile" ;; diff --git a/configure.ac b/configure.ac index 7fddf44..68f0f27 100644 --- a/configure.ac +++ b/configure.ac @@ -2305,6 +2305,7 @@ WINE_CONFIG_MAKEFILE([dlls/midimap/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DL WINE_CONFIG_MAKEFILE([dlls/mlang/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS]) WINE_CONFIG_MAKEFILE([dlls/mlang/tests/Makefile],[dlls/Maketest.rules],[dlls],[ALL_TEST_DIRS],[enable_tests]) WINE_CONFIG_MAKEFILE([dlls/mmdevldr.vxd/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS],[enable_win16]) +WINE_CONFIG_MAKEFILE([dlls/mmsystem.dll16/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS],[enable_win16]) WINE_CONFIG_MAKEFILE([dlls/monodebg.vxd/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS],[enable_win16]) WINE_CONFIG_MAKEFILE([dlls/mountmgr.sys/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS]) WINE_CONFIG_MAKEFILE([dlls/mouse.drv16/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS],[enable_win16]) diff --git a/dlls/Makefile.in b/dlls/Makefile.in index 8dcfdab..92f129f 100644 --- a/dlls/Makefile.in +++ b/dlls/Makefile.in @@ -21,7 +21,6 @@ WIN16_FILES = \ commdlg.dll16 \ gdi.exe16 \ krnl386.exe16 \ - mmsystem.dll16 \ user.exe16 \ wprocs.dll16 @@ -48,9 +47,6 @@ user.exe16: wprocs.dll16: echo "winedos.dll" >$@ -mmsystem.dll16: - echo "winmm.dll" >$@ - # Import libraries STATIC_IMPLIBEXT = $(IMPLIBEXT:def=def.a) diff --git a/dlls/mmsystem.dll16/Makefile.in b/dlls/mmsystem.dll16/Makefile.in new file mode 100644 index 0000000..eb6c6f6 --- /dev/null +++ b/dlls/mmsystem.dll16/Makefile.in @@ -0,0 +1,18 @@ +TOPSRCDIR = @top_srcdir@ +TOPOBJDIR = ../.. +SRCDIR = @srcdir@ +VPATH = @srcdir@ +MODULE = mmsystem.dll16 +IMPORTS = winmm user32 kernel32 + +EXTRADLLFLAGS = -Wb,--subsystem,win16,--main-module,winmm.dll + +C_SRCS = \ + mci16.c \ + message16.c \ + mmio16.c \ + mmsystem.c + +(a)MAKE_DLL_RULES@ + +(a)DEPENDENCIES@ # everything below this line is overwritten by make depend diff --git a/dlls/winmm/mci16.c b/dlls/mmsystem.dll16/mci16.c similarity index 100% rename from dlls/winmm/mci16.c rename to dlls/mmsystem.dll16/mci16.c diff --git a/dlls/winmm/message16.c b/dlls/mmsystem.dll16/message16.c similarity index 100% rename from dlls/winmm/message16.c rename to dlls/mmsystem.dll16/message16.c diff --git a/dlls/winmm/mmio16.c b/dlls/mmsystem.dll16/mmio16.c similarity index 100% rename from dlls/winmm/mmio16.c rename to dlls/mmsystem.dll16/mmio16.c diff --git a/dlls/winmm/mmsystem.c b/dlls/mmsystem.dll16/mmsystem.c similarity index 99% rename from dlls/winmm/mmsystem.c rename to dlls/mmsystem.dll16/mmsystem.c index f258fee..a7578e7 100644 --- a/dlls/winmm/mmsystem.c +++ b/dlls/mmsystem.dll16/mmsystem.c @@ -452,12 +452,12 @@ UINT16 WINAPI auxGetDevCaps16(UINT16 uDeviceID, LPAUXCAPS16 lpCaps, UINT16 uSize ret = auxGetDevCapsA(uDeviceID, &acA, sizeof(acA)); if (ret == MMSYSERR_NOERROR) { AUXCAPS16 ac16; - ac16.wMid = acA.wMid; - ac16.wPid = acA.wPid; - ac16.vDriverVersion = acA.vDriverVersion; - strcpy(ac16.szPname, acA.szPname); - ac16.wTechnology = acA.wTechnology; - ac16.dwSupport = acA.dwSupport; + ac16.wMid = acA.wMid; + ac16.wPid = acA.wPid; + ac16.vDriverVersion = acA.vDriverVersion; + strcpy(ac16.szPname, acA.szPname); + ac16.wTechnology = acA.wTechnology; + ac16.dwSupport = acA.dwSupport; memcpy(lpCaps, &ac16, min(uSize, sizeof(ac16))); } return ret; diff --git a/dlls/winmm/mmsystem.spec b/dlls/mmsystem.dll16/mmsystem.dll16.spec similarity index 100% rename from dlls/winmm/mmsystem.spec rename to dlls/mmsystem.dll16/mmsystem.dll16.spec diff --git a/dlls/winmm/winemm16.h b/dlls/mmsystem.dll16/winemm16.h similarity index 100% rename from dlls/winmm/winemm16.h rename to dlls/mmsystem.dll16/winemm16.h diff --git a/dlls/winmm/Makefile.in b/dlls/winmm/Makefile.in index 416b56b..0848ca5 100644 --- a/dlls/winmm/Makefile.in +++ b/dlls/winmm/Makefile.in @@ -17,15 +17,6 @@ C_SRCS = \ time.c \ winmm.c -C_SRCS16 = \ - mci16.c \ - message16.c \ - mmio16.c \ - mmsystem.c - -SPEC_SRCS16 = \ - mmsystem.spec - RC_SRCS = \ winmm_Cs.rc \ winmm_Da.rc \

1 0