October 2013 - Wine-commits

Alexandre Julliard : ntdll: Add an futimens syscall wrapper for Android.
by Alexandre Julliard 29 Oct '13

29 Oct '13

Module: wine Branch: master Commit: 61770be5f1d3f8c11025ed5ba8fa2902027eb58a URL: http://source.winehq.org/git/wine.git/?a=commit;h=61770be5f1d3f8c11025ed5ba… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Oct 29 10:24:34 2013 +0100 ntdll: Add an futimens syscall wrapper for Android. --- dlls/ntdll/file.c | 12 ++++++++++++ 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/dlls/ntdll/file.c b/dlls/ntdll/file.c index 9dbc8fe..8c8f4cf 100644 --- a/dlls/ntdll/file.c +++ b/dlls/ntdll/file.c @@ -36,6 +36,9 @@ #ifdef HAVE_SYS_PARAM_H # include <sys/param.h> #endif +#ifdef HAVE_SYS_SYSCALL_H +# include <sys/syscall.h> +#endif #ifdef HAVE_SYS_TIME_H # include <sys/time.h> #endif @@ -1654,6 +1657,15 @@ NTSTATUS WINAPI NtSetVolumeInformationFile( return 0; } +#if defined(__ANDROID__) && !defined(HAVE_FUTIMENS) +static int futimens( int fd, const struct timespec spec[2] ) +{ + return syscall( __NR_utimensat, fd, NULL, spec, 0 ); +} +#define UTIME_OMIT ((1 << 30) - 2) +#define HAVE_FUTIMENS +#endif /* __ANDROID__ */ + static NTSTATUS set_file_times( int fd, const LARGE_INTEGER *mtime, const LARGE_INTEGER *atime ) { NTSTATUS status = STATUS_SUCCESS;

1 0

Henri Verbeet : wined3d: Avoid initializing some more variables.
by Alexandre Julliard 29 Oct '13

29 Oct '13

Module: wine Branch: master Commit: 3d226df3b90702507da2529b4f58deff06d3d2a4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3d226df3b90702507da2529b4… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Oct 29 10:28:03 2013 +0100 wined3d: Avoid initializing some more variables. --- dlls/wined3d/state.c | 20 ++++++++++---------- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/dlls/wined3d/state.c b/dlls/wined3d/state.c index 15a0b5a..d31cadf 100644 --- a/dlls/wined3d/state.c +++ b/dlls/wined3d/state.c @@ -821,16 +821,16 @@ static void state_stencil(struct wined3d_context *context, const struct wined3d_ const struct wined3d_gl_info *gl_info = context->gl_info; DWORD onesided_enable; DWORD twosided_enable; - GLint func = GL_ALWAYS; - GLint func_ccw = GL_ALWAYS; - GLint ref = 0; - GLuint mask = 0; - GLint stencilFail = GL_KEEP; - GLint depthFail = GL_KEEP; - GLint stencilPass = GL_KEEP; - GLint stencilFail_ccw = GL_KEEP; - GLint depthFail_ccw = GL_KEEP; - GLint stencilPass_ccw = GL_KEEP; + GLint func; + GLint func_ccw; + GLint ref; + GLuint mask; + GLint stencilFail; + GLint stencilFail_ccw; + GLint stencilPass; + GLint stencilPass_ccw; + GLint depthFail; + GLint depthFail_ccw; /* No stencil test without a stencil buffer. */ if (!state->fb->depth_stencil)

1 0

Henri Verbeet : d3d8/tests: Avoid windowsx.h.
by Alexandre Julliard 29 Oct '13

29 Oct '13

Module: wine Branch: master Commit: f70672336e80a0f727284ec896032e490b120a90 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f70672336e80a0f727284ec89… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Oct 29 10:28:04 2013 +0100 d3d8/tests: Avoid windowsx.h. --- dlls/d3d8/tests/device.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/dlls/d3d8/tests/device.c b/dlls/d3d8/tests/device.c index 9db5335..2e2a0c0 100644 --- a/dlls/d3d8/tests/device.c +++ b/dlls/d3d8/tests/device.c @@ -22,11 +22,13 @@ */ #define COBJMACROS -#include <windowsx.h> #include <initguid.h> #include <d3d8.h> #include "wine/test.h" +#define GET_X_LPARAM(lp) ((int)(short)LOWORD(lp)) +#define GET_Y_LPARAM(lp) ((int)(short)HIWORD(lp)) + static INT screen_width; static INT screen_height;

1 0

Henri Verbeet : d3d8/tests: Use explicit A calls.
by Alexandre Julliard 29 Oct '13

29 Oct '13

Module: wine Branch: master Commit: 45a1aae0a106b192ff854a8fc6a8ec9d59c9cc42 URL: http://source.winehq.org/git/wine.git/?a=commit;h=45a1aae0a106b192ff854a8fc… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Oct 29 10:28:05 2013 +0100 d3d8/tests: Use explicit A calls. --- dlls/d3d8/tests/Makefile.in | 1 - dlls/d3d8/tests/device.c | 65 +++++++++++++++++++++++++---------------- dlls/d3d8/tests/stateblock.c | 8 ++-- dlls/d3d8/tests/visual.c | 10 +++--- 4 files changed, 49 insertions(+), 35 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=45a1aae0a106b192ff854…

1 0

Henri Verbeet : d3drm: Get rid of the color_rgb typedef.
by Alexandre Julliard 29 Oct '13

29 Oct '13

Module: wine Branch: master Commit: 0365d7a93b8b27d9deda435c1f39a011debc1025 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0365d7a93b8b27d9deda435c1… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Oct 29 10:28:06 2013 +0100 d3drm: Get rid of the color_rgb typedef. --- dlls/d3drm/material.c | 10 +++++----- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/dlls/d3drm/material.c b/dlls/d3drm/material.c index 216d913..8eb40b5 100644 --- a/dlls/d3drm/material.c +++ b/dlls/d3drm/material.c @@ -29,21 +29,21 @@ WINE_DEFAULT_DEBUG_CHANNEL(d3drm); -typedef struct +struct color_rgb { D3DVALUE r; D3DVALUE g; D3DVALUE b; -} color_rgb; +}; struct d3drm_material { IDirect3DRMMaterial2 IDirect3DRMMaterial2_iface; LONG ref; - color_rgb emissive; - color_rgb specular; + struct color_rgb emissive; + struct color_rgb specular; D3DVALUE power; - color_rgb ambient; + struct color_rgb ambient; }; static inline struct d3drm_material *impl_from_IDirect3DRMMaterial2(IDirect3DRMMaterial2 *iface)

1 0

Henri Verbeet : d3drm: Get rid of the mesh_group typedef.
by Alexandre Julliard 29 Oct '13

29 Oct '13

Module: wine Branch: master Commit: ff08d70f4a421cbc40bdeab2031c0567e54c38bd URL: http://source.winehq.org/git/wine.git/?a=commit;h=ff08d70f4a421cbc40bdeab20… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Oct 29 10:28:07 2013 +0100 d3drm: Get rid of the mesh_group typedef. --- dlls/d3drm/meshbuilder.c | 15 ++++++++------- 1 files changed, 8 insertions(+), 7 deletions(-) diff --git a/dlls/d3drm/meshbuilder.c b/dlls/d3drm/meshbuilder.c index 0262284..f01d440 100644 --- a/dlls/d3drm/meshbuilder.c +++ b/dlls/d3drm/meshbuilder.c @@ -34,7 +34,8 @@ WINE_DEFAULT_DEBUG_CHANNEL(d3drm); -typedef struct { +struct mesh_group +{ unsigned nb_vertices; D3DRMVERTEX* vertices; unsigned nb_faces; @@ -44,14 +45,14 @@ typedef struct { D3DCOLOR color; IDirect3DRMMaterial2* material; IDirect3DRMTexture3* texture; -} mesh_group; +}; typedef struct { IDirect3DRMMesh IDirect3DRMMesh_iface; LONG ref; DWORD groups_capacity; DWORD nb_groups; - mesh_group* groups; + struct mesh_group *groups; } IDirect3DRMMeshImpl; typedef struct { @@ -2628,7 +2629,7 @@ static HRESULT WINAPI IDirect3DRMMeshImpl_AddGroup(IDirect3DRMMesh* iface, unsigned *face_data, D3DRMGROUPINDEX *return_id) { IDirect3DRMMeshImpl *This = impl_from_IDirect3DRMMesh(iface); - mesh_group* group; + struct mesh_group *group; TRACE("(%p)->(%u,%u,%u,%p,%p)\n", This, vertex_count, face_count, vertex_per_face, face_data, return_id); @@ -2637,18 +2638,18 @@ static HRESULT WINAPI IDirect3DRMMeshImpl_AddGroup(IDirect3DRMMesh* iface, if ((This->nb_groups + 1) > This->groups_capacity) { + struct mesh_group *groups; ULONG new_capacity; - mesh_group* groups; if (!This->groups_capacity) { new_capacity = 16; - groups = HeapAlloc(GetProcessHeap(), 0, new_capacity * sizeof(mesh_group)); + groups = HeapAlloc(GetProcessHeap(), 0, new_capacity * sizeof(*groups)); } else { new_capacity = This->groups_capacity * 2; - groups = HeapReAlloc(GetProcessHeap(), 0, This->groups, new_capacity * sizeof(mesh_group)); + groups = HeapReAlloc(GetProcessHeap(), 0, This->groups, new_capacity * sizeof(*groups)); } if (!groups)

1 0

Alexandre Julliard : explorer: Load the graphics driver specified in the registry instead of relying on gdi32 .
by Alexandre Julliard 29 Oct '13

29 Oct '13

Module: wine Branch: master Commit: 43f042ed9c4702c8f89cc6e05a0c815cfd9d5a49 URL: http://source.winehq.org/git/wine.git/?a=commit;h=43f042ed9c4702c8f89cc6e05… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Oct 29 12:11:52 2013 +0100 explorer: Load the graphics driver specified in the registry instead of relying on gdi32. --- programs/explorer/desktop.c | 112 ++++++++++++++++++++++++++++++++----------- 1 files changed, 83 insertions(+), 29 deletions(-) diff --git a/programs/explorer/desktop.c b/programs/explorer/desktop.c index a3bfb5c..35c3168 100644 --- a/programs/explorer/desktop.c +++ b/programs/explorer/desktop.c @@ -30,7 +30,6 @@ #include <shlobj.h> #include <shellapi.h> -#include "wine/gdi_driver.h" #include "wine/unicode.h" #include "wine/debug.h" #include "explorer_private.h" @@ -40,6 +39,12 @@ WINE_DEFAULT_DEBUG_CHANNEL(explorer); #define DESKTOP_CLASS_ATOM ((LPCWSTR)MAKEINTATOM(32769)) #define DESKTOP_ALL_ACCESS 0x01ff +#ifdef __APPLE__ +static const WCHAR default_driver[] = {'m','a','c',',','x','1','1',0}; +#else +static const WCHAR default_driver[] = {'x','1','1',0}; +#endif + static BOOL using_root; struct launcher @@ -633,34 +638,83 @@ static BOOL get_default_desktop_size( const WCHAR *name, unsigned int *width, un return found; } -static void set_desktop_guid( HWND desktop, GUID *guid, HMODULE driver ) +static HMODULE load_graphics_driver( const GUID *guid ) { static const WCHAR device_keyW[] = { 'S','y','s','t','e','m','\\', 'C','u','r','r','e','n','t','C','o','n','t','r','o','l','S','e','t','\\', 'C','o','n','t','r','o','l','\\', 'V','i','d','e','o','\\', - '{','%','s','}','\\','0','0','0','0',0}; - static const WCHAR driverW[] = {'G','r','a','p','h','i','c','s','D','r','i','v','e','r',0}; - - RPC_WSTR guid_str; - HKEY hkey; + '{','%','0','8','x','-','%','0','4','x','-','%','0','4','x','-', + '%','0','2','x','%','0','2','x','-','%','0','2','x','%','0','2','x','%','0','2','x', + '%','0','2','x','%','0','2','x','%','0','2','x','}','\\','0','0','0','0',0}; + static const WCHAR graphics_driverW[] = {'G','r','a','p','h','i','c','s','D','r','i','v','e','r',0}; + static const WCHAR driversW[] = {'S','o','f','t','w','a','r','e','\\', + 'W','i','n','e','\\','D','r','i','v','e','r','s',0}; + static const WCHAR graphicsW[] = {'G','r','a','p','h','i','c','s',0}; + static const WCHAR drv_formatW[] = {'w','i','n','e','%','s','.','d','r','v',0}; + + WCHAR buffer[MAX_PATH], libname[32], *name, *next; WCHAR key[sizeof(device_keyW)/sizeof(WCHAR) + 39]; + HMODULE module = 0; + HKEY hkey; + char error[80]; + + strcpyW( buffer, default_driver ); + + /* @@ Wine registry key: HKCU\Software\Wine\Drivers */ + if (!RegOpenKeyW( HKEY_CURRENT_USER, driversW, &hkey )) + { + DWORD count = sizeof(buffer); + RegQueryValueExW( hkey, graphicsW, 0, NULL, (LPBYTE)buffer, &count ); + RegCloseKey( hkey ); + } - UuidToStringW( guid, &guid_str ); - sprintfW( key, device_keyW, guid_str ); - RpcStringFreeW( &guid_str ); + name = buffer; + while (name) + { + next = strchrW( name, ',' ); + if (next) *next++ = 0; + + snprintfW( libname, sizeof(libname)/sizeof(WCHAR), drv_formatW, name ); + if ((module = LoadLibraryW( libname )) != 0) break; + switch (GetLastError()) + { + case ERROR_MOD_NOT_FOUND: + strcpy( error, "The graphics driver is missing. Check your build!" ); + break; + case ERROR_DLL_INIT_FAILED: + strcpy( error, "Make sure that your X server is running and that $DISPLAY is set correctly." ); + break; + default: + sprintf( error, "Unknown error (%u).", GetLastError() ); + break; + } + name = next; + } + + if (module) + { + GetModuleFileNameW( module, buffer, MAX_PATH ); + TRACE( "display %s driver %s\n", debugstr_guid(guid), debugstr_w(buffer) ); + } + + sprintfW( key, device_keyW, guid->Data1, guid->Data2, guid->Data3, + guid->Data4[0], guid->Data4[1], guid->Data4[2], guid->Data4[3], + guid->Data4[4], guid->Data4[5], guid->Data4[6], guid->Data4[7] ); if (!RegCreateKeyExW( HKEY_LOCAL_MACHINE, key, 0, NULL, REG_OPTION_VOLATILE, KEY_SET_VALUE, NULL, &hkey, NULL )) { - WCHAR path[MAX_PATH]; - - GetModuleFileNameW( driver, path, MAX_PATH ); - RegSetValueExW( hkey, driverW, 0, REG_SZ, (BYTE *)path, (strlenW(path) + 1) * sizeof(WCHAR) ); + if (module) + RegSetValueExW( hkey, graphics_driverW, 0, REG_SZ, + (BYTE *)buffer, (strlenW(buffer) + 1) * sizeof(WCHAR) ); + else + RegSetValueExA( hkey, "DriverError", 0, REG_SZ, (BYTE *)error, strlen(error) + 1 ); RegCloseKey( hkey ); } - else ERR( "failed to create %s\n", debugstr_w(key) ); + + return module; } static void initialize_display_settings(void) @@ -716,8 +770,8 @@ void manage_desktop( WCHAR *arg ) HDESK desktop = 0; GUID guid; MSG msg; - HDC hdc; HWND hwnd, msg_hwnd; + HMODULE graphics_driver; unsigned int width, height; WCHAR *cmdline = NULL; WCHAR *p = arg; @@ -759,6 +813,7 @@ void manage_desktop( WCHAR *arg ) UuidCreate( &guid ); TRACE( "display guid %s\n", debugstr_guid(&guid) ); + graphics_driver = load_graphics_driver( &guid ); /* create the desktop window */ hwnd = CreateWindowExW( 0, DESKTOP_CLASS_ATOM, NULL, @@ -770,16 +825,8 @@ void manage_desktop( WCHAR *arg ) if (hwnd == GetDesktopWindow()) { - HMODULE shell32, graphics_driver; - void (WINAPI *pShellDDEInit)( BOOL ); - - hdc = GetDC( hwnd ); - graphics_driver = __wine_get_driver_module( hdc ); using_root = !desktop || !create_desktop( graphics_driver, name, width, height ); - ReleaseDC( hwnd, hdc ); - SetWindowLongPtrW( hwnd, GWLP_WNDPROC, (LONG_PTR)desktop_wnd_proc ); - set_desktop_guid( hwnd, &guid, graphics_driver ); SendMessageW( hwnd, WM_SETICON, ICON_BIG, (LPARAM)LoadIconW( 0, MAKEINTRESOURCEW(OIC_WINLOGO))); if (name) set_desktop_window_title( hwnd, name ); SetWindowPos( hwnd, 0, GetSystemMetrics(SM_XVIRTUALSCREEN), GetSystemMetrics(SM_YVIRTUALSCREEN), @@ -789,13 +836,20 @@ void manage_desktop( WCHAR *arg ) ClipCursor( NULL ); initialize_display_settings(); initialize_appbar(); - initialize_systray( graphics_driver, using_root ); - if (!using_root) initialize_launchers( hwnd ); - if ((shell32 = LoadLibraryA( "shell32.dll" )) && - (pShellDDEInit = (void *)GetProcAddress( shell32, (LPCSTR)188))) + if (graphics_driver) { - pShellDDEInit( TRUE ); + HMODULE shell32; + void (WINAPI *pShellDDEInit)( BOOL ); + + initialize_systray( graphics_driver, using_root ); + if (!using_root) initialize_launchers( hwnd ); + + if ((shell32 = LoadLibraryA( "shell32.dll" )) && + (pShellDDEInit = (void *)GetProcAddress( shell32, (LPCSTR)188))) + { + pShellDDEInit( TRUE ); + } } } else

1 0

Alexandre Julliard : user32: Get the graphics driver name from the desktop window registry entry.
by Alexandre Julliard 29 Oct '13

29 Oct '13

Module: wine Branch: master Commit: 010769249c667317cd7b93388bb89fa7c34bad80 URL: http://source.winehq.org/git/wine.git/?a=commit;h=010769249c667317cd7b93388… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Oct 29 12:12:21 2013 +0100 user32: Get the graphics driver name from the desktop window registry entry. --- dlls/user32/driver.c | 66 ++++++++++++++++++++++++++++++++-------------- dlls/user32/misc.c | 3 -- dlls/user32/sysparams.c | 6 ---- 3 files changed, 46 insertions(+), 29 deletions(-) diff --git a/dlls/user32/driver.c b/dlls/user32/driver.c index 192eebe..fdcf853 100644 --- a/dlls/user32/driver.c +++ b/dlls/user32/driver.c @@ -26,20 +26,61 @@ #include "winuser.h" #include "wine/debug.h" #include "wine/gdi_driver.h" +#include "wine/unicode.h" #include "user_private.h" #include "controls.h" +WINE_DEFAULT_DEBUG_CHANNEL(user); + static USER_DRIVER null_driver, lazy_load_driver; const USER_DRIVER *USER_Driver = &lazy_load_driver; -static DWORD driver_load_error; +static char driver_load_error[80]; + +static HMODULE load_desktop_driver( HWND hwnd ) +{ + static const WCHAR display_device_guid_propW[] = { + '_','_','w','i','n','e','_','d','i','s','p','l','a','y','_', + 'd','e','v','i','c','e','_','g','u','i','d',0 }; + static const WCHAR key_pathW[] = { + 'S','y','s','t','e','m','\\', + 'C','u','r','r','e','n','t','C','o','n','t','r','o','l','S','e','t','\\', + 'C','o','n','t','r','o','l','\\', + 'V','i','d','e','o','\\','{',0}; + static const WCHAR displayW[] = {'}','\\','0','0','0','0',0}; + static const WCHAR driverW[] = {'G','r','a','p','h','i','c','s','D','r','i','v','e','r',0}; + HMODULE ret = 0; + HKEY hkey; + DWORD size; + WCHAR path[MAX_PATH]; + WCHAR key[(sizeof(key_pathW) + sizeof(displayW)) / sizeof(WCHAR) + 40]; + UINT guid_atom = HandleToULong( GetPropW( hwnd, display_device_guid_propW )); + + strcpy( driver_load_error, "The explorer process failed to start." ); /* default error */ + + memcpy( key, key_pathW, sizeof(key_pathW) ); + if (!GlobalGetAtomNameW( guid_atom, key + strlenW(key), 40 )) return 0; + strcatW( key, displayW ); + if (RegOpenKeyW( HKEY_LOCAL_MACHINE, key, &hkey )) return 0; + size = sizeof(path); + if (!RegQueryValueExW( hkey, driverW, NULL, NULL, (BYTE *)path, &size )) + { + if (!(ret = LoadLibraryW( path ))) ERR( "failed to load %s\n", debugstr_w(path) ); + TRACE( "%s %p\n", debugstr_w(path), ret ); + } + else + { + size = sizeof(driver_load_error); + RegQueryValueExA( hkey, "DriverError", NULL, NULL, (BYTE *)driver_load_error, &size ); + } + RegCloseKey( hkey ); + return ret; +} /* load the graphics driver */ static const USER_DRIVER *load_driver(void) { - static const WCHAR displayW[] = {'D','I','S','P','L','A','Y',0}; - HDC hdc; void *ptr; HMODULE graphics_driver; USER_DRIVER *driver, *prev; @@ -47,9 +88,7 @@ static const USER_DRIVER *load_driver(void) driver = HeapAlloc( GetProcessHeap(), 0, sizeof(*driver) ); *driver = null_driver; - hdc = CreateDCW( displayW, NULL, NULL, NULL ); - - graphics_driver = __wine_get_driver_module( hdc ); + graphics_driver = load_desktop_driver( GetDesktopWindow() ); if (graphics_driver) { #define GET_USER_FUNC(name) \ @@ -109,7 +148,6 @@ static const USER_DRIVER *load_driver(void) GET_USER_FUNC(SystemParametersInfo); #undef GET_USER_FUNC } - else driver_load_error = GetLastError(); prev = InterlockedCompareExchangePointer( (void **)&USER_Driver, driver, &lazy_load_driver ); if (prev != &lazy_load_driver) @@ -122,7 +160,6 @@ static const USER_DRIVER *load_driver(void) register_builtin_classes(); - DeleteDC( hdc ); return driver; } @@ -304,18 +341,7 @@ static BOOL CDECL nulldrv_CreateWindow( HWND hwnd ) if (warned++) return FALSE; MESSAGE( "Application tried to create a window, but no driver could be loaded.\n"); - switch (driver_load_error) - { - case ERROR_MOD_NOT_FOUND: - MESSAGE( "The graphics driver is missing. Check your build!\n" ); - break; - case ERROR_DLL_INIT_FAILED: - MESSAGE( "Make sure that your X server is running and that $DISPLAY is set correctly.\n" ); - break; - default: - MESSAGE( "Unknown error (%d).\n", driver_load_error ); - } - + if (driver_load_error[0]) MESSAGE( "%s\n", driver_load_error ); return FALSE; } diff --git a/dlls/user32/misc.c b/dlls/user32/misc.c index 9177f56..ad3c788 100644 --- a/dlls/user32/misc.c +++ b/dlls/user32/misc.c @@ -441,9 +441,6 @@ BOOL WINAPI GetMonitorInfoW(HMONITOR hMonitor, LPMONITORINFO lpMonitorInfo) */ BOOL WINAPI EnumDisplayMonitors( HDC hdc, LPRECT rect, MONITORENUMPROC proc, LPARAM lp ) { - /* make sure the desktop window exists */ - GetDesktopWindow(); - return USER_Driver->pEnumDisplayMonitors( hdc, rect, proc, lp ); } diff --git a/dlls/user32/sysparams.c b/dlls/user32/sysparams.c index bdfed38..4b510f2 100644 --- a/dlls/user32/sysparams.c +++ b/dlls/user32/sysparams.c @@ -2792,9 +2792,6 @@ LONG WINAPI ChangeDisplaySettingsExA( LPCSTR devname, LPDEVMODEA devmode, HWND h LONG WINAPI ChangeDisplaySettingsExW( LPCWSTR devname, LPDEVMODEW devmode, HWND hwnd, DWORD flags, LPVOID lparam ) { - /* make sure the desktop window is created before mode changing */ - GetDesktopWindow(); - return USER_Driver->pChangeDisplaySettingsEx( devname, devmode, hwnd, flags, lparam ); } @@ -2866,9 +2863,6 @@ BOOL WINAPI EnumDisplaySettingsExA(LPCSTR lpszDeviceName, DWORD iModeNum, BOOL WINAPI EnumDisplaySettingsExW(LPCWSTR lpszDeviceName, DWORD iModeNum, LPDEVMODEW lpDevMode, DWORD dwFlags) { - /* make sure the desktop window is created before mode enumeration */ - GetDesktopWindow(); - return USER_Driver->pEnumDisplaySettingsEx(lpszDeviceName, iModeNum, lpDevMode, dwFlags); }

1 0

Alexandre Julliard : gdi32: Allow user32 to specify the display driver to use.
by Alexandre Julliard 29 Oct '13

29 Oct '13

Module: wine Branch: master Commit: 9675aa173a779eba5e97ee4bbbde6a6cfeced179 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9675aa173a779eba5e97ee4bb… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Oct 29 12:39:46 2013 +0100 gdi32: Allow user32 to specify the display driver to use. --- dlls/gdi32/driver.c | 17 +++++++++++++++++ dlls/gdi32/gdi32.spec | 1 + dlls/user32/driver.c | 1 + include/wine/gdi_driver.h | 1 + 4 files changed, 20 insertions(+), 0 deletions(-) diff --git a/dlls/gdi32/driver.c b/dlls/gdi32/driver.c index f625f32..73d11da 100644 --- a/dlls/gdi32/driver.c +++ b/dlls/gdi32/driver.c @@ -223,6 +223,23 @@ HMODULE CDECL __wine_get_driver_module( HDC hdc ) } +/*********************************************************************** + * __wine_set_display_driver_module (GDI32.@) + */ +void CDECL __wine_set_display_driver( HMODULE module ) +{ + struct graphics_driver *driver; + + if (!(driver = create_driver( module ))) + { + ERR( "Could not create graphics driver\n" ); + ExitProcess(1); + } + if (InterlockedCompareExchangePointer( (void **)&display_driver, driver, NULL )) + HeapFree( GetProcessHeap(), 0, driver ); +} + + static INT nulldrv_AbortDoc( PHYSDEV dev ) { return 0; diff --git a/dlls/gdi32/gdi32.spec b/dlls/gdi32/gdi32.spec index 65f1d0c..be6410d 100644 --- a/dlls/gdi32/gdi32.spec +++ b/dlls/gdi32/gdi32.spec @@ -515,6 +515,7 @@ # Graphics drivers @ cdecl __wine_get_driver_module(long) +@ cdecl __wine_set_display_driver(long) # OpenGL @ cdecl __wine_get_wgl_driver(long long) diff --git a/dlls/user32/driver.c b/dlls/user32/driver.c index fdcf853..37de424 100644 --- a/dlls/user32/driver.c +++ b/dlls/user32/driver.c @@ -158,6 +158,7 @@ static const USER_DRIVER *load_driver(void) } else LdrAddRefDll( 0, graphics_driver ); + __wine_set_display_driver( graphics_driver ); register_builtin_classes(); return driver; diff --git a/include/wine/gdi_driver.h b/include/wine/gdi_driver.h index 83f4885..ad9c7c4 100644 --- a/include/wine/gdi_driver.h +++ b/include/wine/gdi_driver.h @@ -277,6 +277,7 @@ extern void CDECL __wine_make_gdi_object_system( HGDIOBJ handle, BOOL set ); extern void CDECL __wine_set_visible_region( HDC hdc, HRGN hrgn, const RECT *vis_rect, const RECT *device_rect, struct window_surface *surface ); extern HMODULE CDECL __wine_get_driver_module( HDC hdc ); +extern void CDECL __wine_set_display_driver( HMODULE module ); extern struct opengl_funcs * CDECL __wine_get_wgl_driver( HDC hdc, UINT version ); #endif /* __WINE_WINE_GDI_DRIVER_H */

1 0

Alexandre Julliard : gdi32: Trigger creation of the desktop window when the display driver is needed.
by Alexandre Julliard 29 Oct '13

29 Oct '13

Module: wine Branch: master Commit: 74efeab238558902d1f3f3e8c31035f52d9920d4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=74efeab238558902d1f3f3e8c… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Oct 29 12:43:30 2013 +0100 gdi32: Trigger creation of the desktop window when the display driver is needed. --- dlls/gdi32/driver.c | 67 +++++--------------------------------------- dlls/gdi32/gdi32.spec | 1 - include/wine/gdi_driver.h | 1 - 3 files changed, 8 insertions(+), 61 deletions(-) diff --git a/dlls/gdi32/driver.c b/dlls/gdi32/driver.c index 73d11da..99ac146 100644 --- a/dlls/gdi32/driver.c +++ b/dlls/gdi32/driver.c @@ -28,7 +28,6 @@ #include <stdio.h> #include "windef.h" #include "winbase.h" -#include "winreg.h" #include "ddrawgdi.h" #include "wine/winbase16.h" @@ -48,7 +47,6 @@ struct graphics_driver static struct list drivers = LIST_INIT( drivers ); static struct graphics_driver *display_driver; -static DWORD display_driver_load_error; const struct gdi_dc_funcs *font_driver = NULL; @@ -101,48 +99,18 @@ static struct graphics_driver *create_driver( HMODULE module ) */ static const struct gdi_dc_funcs *get_display_driver( HMODULE *module_ret ) { - struct graphics_driver *driver; - char buffer[MAX_PATH], libname[32], *name, *next; - HMODULE module = 0; - HKEY hkey; - - if (display_driver) goto done; - - strcpy( buffer, default_driver ); - /* @@ Wine registry key: HKCU\Software\Wine\Drivers */ - if (!RegOpenKeyA( HKEY_CURRENT_USER, "Software\\Wine\\Drivers", &hkey )) - { - DWORD type, count = sizeof(buffer); - RegQueryValueExA( hkey, "Graphics", 0, &type, (LPBYTE) buffer, &count ); - RegCloseKey( hkey ); - } - - name = buffer; - while (name) + if (!display_driver) { - next = strchr( name, ',' ); - if (next) *next++ = 0; + HMODULE user32 = LoadLibraryA( "user32.dll" ); + HWND (WINAPI *pGetDesktopWindow)(void) = (void *)GetProcAddress( user32, "GetDesktopWindow" ); - snprintf( libname, sizeof(libname), "wine%s.drv", name ); - if ((module = LoadLibraryA( libname )) != 0) break; - name = next; + if (!pGetDesktopWindow() || !display_driver) + { + WARN( "failed to load the display driver, falling back to null driver\n" ); + __wine_set_display_driver( 0 ); + } } - if (!module) display_driver_load_error = GetLastError(); - - if (!(driver = create_driver( module ))) - { - MESSAGE( "Could not create graphics driver '%s'\n", buffer ); - FreeLibrary( module ); - ExitProcess(1); - } - if (InterlockedCompareExchangePointer( (void **)&display_driver, driver, NULL )) - { - /* somebody beat us to it */ - FreeLibrary( driver->module ); - HeapFree( GetProcessHeap(), 0, driver ); - } -done: *module_ret = display_driver->module; return display_driver->funcs; } @@ -205,25 +173,6 @@ done: /*********************************************************************** - * __wine_get_driver_module (GDI32.@) - */ -HMODULE CDECL __wine_get_driver_module( HDC hdc ) -{ - DC *dc; - HMODULE ret = 0; - - if ((dc = get_dc_ptr( hdc ))) - { - ret = dc->module; - release_dc_ptr( dc ); - if (!ret) SetLastError( display_driver_load_error ); - } - else SetLastError( ERROR_INVALID_HANDLE ); - return ret; -} - - -/*********************************************************************** * __wine_set_display_driver_module (GDI32.@) */ void CDECL __wine_set_display_driver( HMODULE module ) diff --git a/dlls/gdi32/gdi32.spec b/dlls/gdi32/gdi32.spec index be6410d..6ebad3c 100644 --- a/dlls/gdi32/gdi32.spec +++ b/dlls/gdi32/gdi32.spec @@ -514,7 +514,6 @@ @ cdecl __wine_set_visible_region(long long ptr ptr ptr) # Graphics drivers -@ cdecl __wine_get_driver_module(long) @ cdecl __wine_set_display_driver(long) # OpenGL diff --git a/include/wine/gdi_driver.h b/include/wine/gdi_driver.h index ad9c7c4..06e4caa 100644 --- a/include/wine/gdi_driver.h +++ b/include/wine/gdi_driver.h @@ -276,7 +276,6 @@ WINGDIAPI WORD WINAPI SetHookFlags(HDC,WORD); extern void CDECL __wine_make_gdi_object_system( HGDIOBJ handle, BOOL set ); extern void CDECL __wine_set_visible_region( HDC hdc, HRGN hrgn, const RECT *vis_rect, const RECT *device_rect, struct window_surface *surface ); -extern HMODULE CDECL __wine_get_driver_module( HDC hdc ); extern void CDECL __wine_set_display_driver( HMODULE module ); extern struct opengl_funcs * CDECL __wine_get_wgl_driver( HDC hdc, UINT version );

1 0