September 2011 - Wine-commits

Octavian Voicu : opencl: Remove reference to clSetCommandQueueProperty and print fixme, saying function is deprecated.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: f8877ea337d644e6514d2b738862ec5fed1d8751 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f8877ea337d644e6514d2b738… Author: Octavian Voicu <octavian.voicu(a)gmail.com> Date: Thu Sep 1 09:22:41 2011 +0300 opencl: Remove reference to clSetCommandQueueProperty and print fixme, saying function is deprecated. --- dlls/opencl/opencl.c | 6 ++---- 1 files changed, 2 insertions(+), 4 deletions(-) diff --git a/dlls/opencl/opencl.c b/dlls/opencl/opencl.c index 03acb48..9b466ec 100644 --- a/dlls/opencl/opencl.c +++ b/dlls/opencl/opencl.c @@ -276,10 +276,8 @@ cl_int WINAPI wine_clGetCommandQueueInfo(cl_command_queue command_queue, cl_comm cl_int WINAPI wine_clSetCommandQueueProperty(cl_command_queue command_queue, cl_command_queue_properties properties, cl_bool enable, cl_command_queue_properties * old_properties) { - cl_int ret; - TRACE("%p, 0x%lx, %d, %p\n", command_queue, (long unsigned int)properties, enable, old_properties); - ret = clSetCommandQueueProperty(command_queue, properties, enable, old_properties); - return ret; + FIXME("(%p, 0x%lx, %d, %p): deprecated\n", command_queue, (long unsigned int)properties, enable, old_properties); + return CL_INVALID_QUEUE_PROPERTIES; }

1 0

Octavian Voicu : shell32/tests: Add a todo for two recycle bin tests in case no item was added.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: 9b29b10e2325b4cc07a82258ddac74dc2ec1cb3e URL: http://source.winehq.org/git/wine.git/?a=commit;h=9b29b10e2325b4cc07a82258d… Author: Octavian Voicu <octavian.voicu(a)gmail.com> Date: Thu Sep 1 09:53:02 2011 +0300 shell32/tests: Add a todo for two recycle bin tests in case no item was added. --- dlls/shell32/tests/recyclebin.c | 9 +++++++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/dlls/shell32/tests/recyclebin.c b/dlls/shell32/tests/recyclebin.c index 2fa4d3e..391e025 100644 --- a/dlls/shell32/tests/recyclebin.c +++ b/dlls/shell32/tests/recyclebin.c @@ -90,8 +90,13 @@ static void test_query_recyclebin(void) ok(!pSHFileOperationA(&shfo), "Deletion was not successful\n"); hr = pSHQueryRecycleBinA(buf,&info2); ok(hr == S_OK, "SHQueryRecycleBinW failed with error 0x%x\n", hr); - ok(info2.i64Size==info1.i64Size+written,"Expected recycle bin to have 0x%s bytes\n",str_from_int64(info1.i64Size+written)); - ok(info2.i64NumItems==info1.i64NumItems+1,"Expected recycle bin to have 0x%s items\n",str_from_int64(info1.i64NumItems+1)); + if(info2.i64Size!=info1.i64Size || info2.i64NumItems!=info1.i64NumItems) { + ok(info2.i64Size==info1.i64Size+written,"Expected recycle bin to have 0x%s bytes\n",str_from_int64(info1.i64Size+written)); + ok(info2.i64NumItems==info1.i64NumItems+1,"Expected recycle bin to have 0x%s items\n",str_from_int64(info1.i64NumItems+1)); + } else todo_wine { + ok(info2.i64Size==info1.i64Size+written,"Expected recycle bin to have 0x%s bytes\n",str_from_int64(info1.i64Size+written)); + ok(info2.i64NumItems==info1.i64NumItems+1,"Expected recycle bin to have 0x%s items\n",str_from_int64(info1.i64NumItems+1)); + } }

1 0

Alexandre Julliard : wininet/tests: Remove an invalid FTP test.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: 006bb26b8ce4f7dcccc0c7eca1f30f939b070062 URL: http://source.winehq.org/git/wine.git/?a=commit;h=006bb26b8ce4f7dcccc0c7eca… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Sep 1 15:21:01 2011 +0200 wininet/tests: Remove an invalid FTP test. --- dlls/wininet/tests/ftp.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/dlls/wininet/tests/ftp.c b/dlls/wininet/tests/ftp.c index a483dc9..7fbbe41 100644 --- a/dlls/wininet/tests/ftp.c +++ b/dlls/wininet/tests/ftp.c @@ -726,8 +726,7 @@ static void test_command(HINTERNET hFtp, HINTERNET hConnect) { TRUE, ERROR_SUCCESS, "SIZE\t/welcome.msg" }, { TRUE, ERROR_SUCCESS, "SIZE /welcome.msg" }, { FALSE, ERROR_INTERNET_EXTENDED_ERROR, "PWD /welcome.msg" }, - { TRUE, ERROR_SUCCESS, "PWD" }, - { TRUE, ERROR_SUCCESS, "PWD\r\n" } + { TRUE, ERROR_SUCCESS, "PWD" } }; if (!pFtpCommandA)

1 0

Bernhard Loos : ntdll: Provide a timestamp debug channel, which prints a millisecond resolution timestamp in front of debug output.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: 23e1777054f2a801ce14991793cf68b577717deb URL: http://source.winehq.org/git/wine.git/?a=commit;h=23e1777054f2a801ce1499179… Author: Bernhard Loos <bernhardloos(a)googlemail.com> Date: Thu Sep 1 09:21:06 2011 +0200 ntdll: Provide a timestamp debug channel, which prints a millisecond resolution timestamp in front of debug output. --- dlls/ntdll/debugtools.c | 8 +++++++- dlls/ntdll/relay.c | 16 ++++++++++++++++ 2 files changed, 23 insertions(+), 1 deletions(-) diff --git a/dlls/ntdll/debugtools.c b/dlls/ntdll/debugtools.c index cb08285..dfb44af 100644 --- a/dlls/ntdll/debugtools.c +++ b/dlls/ntdll/debugtools.c @@ -39,6 +39,7 @@ #include "ntdll_misc.h" WINE_DECLARE_DEBUG_CHANNEL(tid); +WINE_DECLARE_DEBUG_CHANNEL(timestamp); static struct __wine_debug_functions default_funcs; @@ -164,8 +165,13 @@ static int NTDLL_dbg_vlog( enum __wine_debug_class cls, struct __wine_debug_chan /* only print header if we are at the beginning of the line */ if (info->out_pos == info->output || info->out_pos[-1] == '\n') { + if (TRACE_ON(timestamp)) + { + ULONG ticks = NtGetTickCount(); + ret = wine_dbg_printf( "%3u.%03u:", ticks / 1000, ticks % 1000 ); + } if (TRACE_ON(tid)) - ret = wine_dbg_printf( "%04x:", GetCurrentThreadId() ); + ret += wine_dbg_printf( "%04x:", GetCurrentThreadId() ); if (cls < sizeof(classes)/sizeof(classes[0])) ret += wine_dbg_printf( "%s:%s:%s ", classes[cls], channel->name, function ); } diff --git a/dlls/ntdll/relay.c b/dlls/ntdll/relay.c index 4855a00..3bf7791 100644 --- a/dlls/ntdll/relay.c +++ b/dlls/ntdll/relay.c @@ -37,6 +37,7 @@ #include "wine/debug.h" WINE_DEFAULT_DEBUG_CHANNEL(relay); +WINE_DECLARE_DEBUG_CHANNEL(timestamp); #if defined(__i386__) || defined(__x86_64__) @@ -396,6 +397,13 @@ __ASM_GLOBAL_FUNC( call_entry_point, #endif +static void print_timestamp(void) +{ + ULONG ticks = NtGetTickCount(); + DPRINTF( "%3u.%03u:", ticks / 1000, ticks % 1000 ); +} + + /*********************************************************************** * relay_call * @@ -414,6 +422,8 @@ static LONGLONG WINAPI relay_call( struct relay_descr *descr, unsigned int idx, ret = call_entry_point( entry_point->orig_func, nb_args, stack + 1, flags ); else { + if (TRACE_ON(timestamp)) + print_timestamp(); if (entry_point->name) DPRINTF( "%04x:Call %s.%s(", GetCurrentThreadId(), data->dllname, entry_point->name ); else @@ -423,6 +433,8 @@ static LONGLONG WINAPI relay_call( struct relay_descr *descr, unsigned int idx, ret = call_entry_point( entry_point->orig_func, nb_args, stack + 1, flags ); + if (TRACE_ON(timestamp)) + print_timestamp(); if (entry_point->name) DPRINTF( "%04x:Ret %s.%s()", GetCurrentThreadId(), data->dllname, entry_point->name ); else @@ -933,6 +945,8 @@ void WINAPI __regs_SNOOP_Entry( CONTEXT *context ) context->Eip = (DWORD)fun->origfun; + if (TRACE_ON(timestamp)) + print_timestamp(); if (fun->name) DPRINTF("%04x:CALL %s.%s(",GetCurrentThreadId(),dll->name,fun->name); else DPRINTF("%04x:CALL %s.%d(",GetCurrentThreadId(),dll->name,dll->ordbase+ordinal); if (fun->nrofargs>0) { @@ -967,6 +981,8 @@ void WINAPI __regs_SNOOP_Return( CONTEXT *context ) if (ret->dll->funs[ret->ordinal].nrofargs<0) ret->dll->funs[ret->ordinal].nrofargs=(context->Esp - ret->origESP-4)/4; context->Eip = (DWORD)ret->origreturn; + if (TRACE_ON(timestamp)) + print_timestamp(); if (ret->args) { int i,max;

1 0

Henri Verbeet : wined3d: Merge the wined3d_surface_set_mem() implementations.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: b587b85394114383b0310343821979f507236c36 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b587b85394114383b03103438… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Aug 31 20:40:42 2011 +0200 wined3d: Merge the wined3d_surface_set_mem() implementations. --- dlls/wined3d/surface.c | 173 ++++++++++++++------------------------- dlls/wined3d/wined3d_private.h | 1 - 2 files changed, 62 insertions(+), 112 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 7c31a47..cd01ae6 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -1759,67 +1759,6 @@ HRESULT CDECL wined3d_surface_bltfast(struct wined3d_surface *dst_surface, DWORD return wined3d_surface_blt(dst_surface, &dst_rect, src_surface, &src_rect, flags, NULL, WINED3DTEXF_POINT); } -static HRESULT surface_set_mem(struct wined3d_surface *surface, void *mem) -{ - TRACE("surface %p, mem %p.\n", surface, mem); - - if (mem && mem != surface->resource.allocatedMemory) - { - void *release = NULL; - - /* Do I have to copy the old surface content? */ - if (surface->flags & SFLAG_DIBSECTION) - { - SelectObject(surface->hDC, surface->dib.holdbitmap); - DeleteDC(surface->hDC); - /* Release the DIB section. */ - DeleteObject(surface->dib.DIBsection); - surface->dib.bitmap_data = NULL; - surface->resource.allocatedMemory = NULL; - surface->hDC = NULL; - surface->flags &= ~SFLAG_DIBSECTION; - } - else if (!(surface->flags & SFLAG_USERPTR)) - { - release = surface->resource.heapMemory; - surface->resource.heapMemory = NULL; - } - surface->resource.allocatedMemory = mem; - surface->flags |= SFLAG_USERPTR; - - /* Now the surface memory is most up do date. Invalidate drawable and texture. */ - surface_modify_location(surface, SFLAG_INSYSMEM, TRUE); - - /* For client textures OpenGL has to be notified. */ - if (surface->flags & SFLAG_CLIENT) - surface_release_client_storage(surface); - - /* Now free the old memory if any. */ - HeapFree(GetProcessHeap(), 0, release); - } - else if (surface->flags & SFLAG_USERPTR) - { - /* HeapMemory should be NULL already. */ - if (surface->resource.heapMemory) - ERR("User pointer surface has heap memory allocated.\n"); - - if (!mem) - { - surface->resource.allocatedMemory = NULL; - surface->flags &= ~(SFLAG_USERPTR | SFLAG_INSYSMEM); - - if (surface->flags & SFLAG_CLIENT) - surface_release_client_storage(surface); - - surface_prepare_system_memory(surface); - } - - surface_modify_location(surface, SFLAG_INSYSMEM, TRUE); - } - - return WINED3D_OK; -} - /* Context activation is done by the caller. */ static void surface_remove_pbo(struct wined3d_surface *surface, const struct wined3d_gl_info *gl_info) { @@ -1946,7 +1885,6 @@ static const struct wined3d_surface_ops surface_ops = surface_unmap, surface_getdc, surface_flip, - surface_set_mem, }; /***************************************************************************** @@ -2129,53 +2067,6 @@ static HRESULT gdi_surface_flip(struct wined3d_surface *surface, struct wined3d_ return WINED3D_OK; } -static HRESULT gdi_surface_set_mem(struct wined3d_surface *surface, void *mem) -{ - TRACE("surface %p, mem %p.\n", surface, mem); - - /* Render targets depend on their hdc, and we can't create an hdc on a user pointer. */ - if (surface->resource.usage & WINED3DUSAGE_RENDERTARGET) - { - ERR("Not supported on render targets.\n"); - return WINED3DERR_INVALIDCALL; - } - - if (mem && mem != surface->resource.allocatedMemory) - { - void *release = NULL; - - /* Do I have to copy the old surface content? */ - if (surface->flags & SFLAG_DIBSECTION) - { - SelectObject(surface->hDC, surface->dib.holdbitmap); - DeleteDC(surface->hDC); - /* Release the DIB section. */ - DeleteObject(surface->dib.DIBsection); - surface->dib.bitmap_data = NULL; - surface->resource.allocatedMemory = NULL; - surface->hDC = NULL; - surface->flags &= ~SFLAG_DIBSECTION; - } - else if (!(surface->flags & SFLAG_USERPTR)) - { - release = surface->resource.allocatedMemory; - } - surface->resource.allocatedMemory = mem; - surface->flags |= SFLAG_USERPTR | SFLAG_INSYSMEM; - - /* Now free the old memory, if any. */ - HeapFree(GetProcessHeap(), 0, release); - } - else if (surface->flags & SFLAG_USERPTR) - { - /* Map() and GetDC() will re-create the dib section and allocated memory. */ - surface->resource.allocatedMemory = NULL; - surface->flags &= ~SFLAG_USERPTR; - } - - return WINED3D_OK; -} - static const struct wined3d_surface_ops gdi_surface_ops = { gdi_surface_private_setup, @@ -2187,7 +2078,6 @@ static const struct wined3d_surface_ops gdi_surface_ops = gdi_surface_unmap, gdi_surface_getdc, gdi_surface_flip, - gdi_surface_set_mem, }; void surface_set_texture_name(struct wined3d_surface *surface, GLuint new_name, BOOL srgb) @@ -3159,7 +3049,68 @@ HRESULT CDECL wined3d_surface_set_mem(struct wined3d_surface *surface, void *mem return WINED3DERR_INVALIDCALL; } - return surface->surface_ops->surface_set_mem(surface, mem); + /* Render targets depend on their hdc, and we can't create an hdc on a user pointer. */ + if (surface->resource.usage & WINED3DUSAGE_RENDERTARGET) + { + ERR("Not supported on render targets.\n"); + return WINED3DERR_INVALIDCALL; + } + + if (mem && mem != surface->resource.allocatedMemory) + { + void *release = NULL; + + /* Do I have to copy the old surface content? */ + if (surface->flags & SFLAG_DIBSECTION) + { + SelectObject(surface->hDC, surface->dib.holdbitmap); + DeleteDC(surface->hDC); + /* Release the DIB section. */ + DeleteObject(surface->dib.DIBsection); + surface->dib.bitmap_data = NULL; + surface->resource.allocatedMemory = NULL; + surface->hDC = NULL; + surface->flags &= ~SFLAG_DIBSECTION; + } + else if (!(surface->flags & SFLAG_USERPTR)) + { + release = surface->resource.heapMemory; + surface->resource.heapMemory = NULL; + } + surface->resource.allocatedMemory = mem; + surface->flags |= SFLAG_USERPTR; + + /* Now the surface memory is most up do date. Invalidate drawable and texture. */ + surface_modify_location(surface, SFLAG_INSYSMEM, TRUE); + + /* For client textures OpenGL has to be notified. */ + if (surface->flags & SFLAG_CLIENT) + surface_release_client_storage(surface); + + /* Now free the old memory if any. */ + HeapFree(GetProcessHeap(), 0, release); + } + else if (surface->flags & SFLAG_USERPTR) + { + /* HeapMemory should be NULL already. */ + if (surface->resource.heapMemory) + ERR("User pointer surface has heap memory allocated.\n"); + + if (!mem) + { + surface->resource.allocatedMemory = NULL; + surface->flags &= ~(SFLAG_USERPTR | SFLAG_INSYSMEM); + + if (surface->flags & SFLAG_CLIENT) + surface_release_client_storage(surface); + + surface_prepare_system_memory(surface); + } + + surface_modify_location(surface, SFLAG_INSYSMEM, TRUE); + } + + return WINED3D_OK; } HRESULT CDECL wined3d_surface_set_overlay_position(struct wined3d_surface *surface, LONG x, LONG y) diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index a474c33..ac21a89 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1998,7 +1998,6 @@ struct wined3d_surface_ops void (*surface_unmap)(struct wined3d_surface *surface); HRESULT (*surface_getdc)(struct wined3d_surface *surface); HRESULT (*surface_flip)(struct wined3d_surface *surface, struct wined3d_surface *override); - HRESULT (*surface_set_mem)(struct wined3d_surface *surface, void *mem); }; struct wined3d_surface

1 0

Henri Verbeet : wined3d: Merge the wined3d_surface_flip() implementations.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: 55ce85ad291decf7856875cc6eeae62458514c9d URL: http://source.winehq.org/git/wine.git/?a=commit;h=55ce85ad291decf7856875cc6… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Aug 31 20:40:43 2011 +0200 wined3d: Merge the wined3d_surface_flip() implementations. --- dlls/wined3d/surface.c | 57 +++++++++++++-------------------------- dlls/wined3d/wined3d_private.h | 1 - 2 files changed, 19 insertions(+), 39 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index cd01ae6..4d7e35e 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -1057,31 +1057,6 @@ static HRESULT surface_getdc(struct wined3d_surface *surface) return hr; } -static HRESULT surface_flip(struct wined3d_surface *surface, struct wined3d_surface *override) -{ - TRACE("surface %p, override %p.\n", surface, override); - - /* Flipping is only supported on render targets and overlays. */ - if (!(surface->resource.usage & (WINED3DUSAGE_RENDERTARGET | WINED3DUSAGE_OVERLAY))) - { - WARN("Tried to flip a non-render target, non-overlay surface.\n"); - return WINEDDERR_NOTFLIPPABLE; - } - - if (surface->resource.usage & WINED3DUSAGE_OVERLAY) - { - flip_surface(surface, override); - - /* Update the overlay if it is visible */ - if (surface->overlay_dest) - return surface->surface_ops->surface_draw_overlay(surface); - else - return WINED3D_OK; - } - - return WINED3D_OK; -} - static BOOL surface_is_full_rect(const struct wined3d_surface *surface, const RECT *r) { if ((r->left && r->right) || abs(r->right - r->left) != surface->resource.width) @@ -1884,7 +1859,6 @@ static const struct wined3d_surface_ops surface_ops = surface_map, surface_unmap, surface_getdc, - surface_flip, }; /***************************************************************************** @@ -2060,13 +2034,6 @@ static HRESULT gdi_surface_getdc(struct wined3d_surface *surface) return hr; } -static HRESULT gdi_surface_flip(struct wined3d_surface *surface, struct wined3d_surface *override) -{ - TRACE("surface %p, override %p.\n", surface, override); - - return WINED3D_OK; -} - static const struct wined3d_surface_ops gdi_surface_ops = { gdi_surface_private_setup, @@ -2077,7 +2044,6 @@ static const struct wined3d_surface_ops gdi_surface_ops = gdi_surface_map, gdi_surface_unmap, gdi_surface_getdc, - gdi_surface_flip, }; void surface_set_texture_name(struct wined3d_surface *surface, GLuint new_name, BOOL srgb) @@ -3779,10 +3745,11 @@ HRESULT CDECL wined3d_surface_releasedc(struct wined3d_surface *surface, HDC dc) HRESULT CDECL wined3d_surface_flip(struct wined3d_surface *surface, struct wined3d_surface *override, DWORD flags) { struct wined3d_swapchain *swapchain; - HRESULT hr; TRACE("surface %p, override %p, flags %#x.\n", surface, override, flags); + /* FIXME: This will also prevent overlay flips, since overlays aren't on + * a swapchain either. */ if (surface->container.type != WINED3D_CONTAINER_SWAPCHAIN) { ERR("Flipped surface is not on a swapchain.\n"); @@ -3790,9 +3757,23 @@ HRESULT CDECL wined3d_surface_flip(struct wined3d_surface *surface, struct wined } swapchain = surface->container.u.swapchain; - hr = surface->surface_ops->surface_flip(surface, override); - if (FAILED(hr)) - return hr; + /* Flipping is only supported on render targets and overlays. */ + if (!(surface->resource.usage & (WINED3DUSAGE_RENDERTARGET | WINED3DUSAGE_OVERLAY))) + { + WARN("Tried to flip a non-render target, non-overlay surface.\n"); + return WINEDDERR_NOTFLIPPABLE; + } + + if (surface->resource.usage & WINED3DUSAGE_OVERLAY) + { + flip_surface(surface, override); + + /* Update the overlay if it is visible */ + if (surface->overlay_dest) + return surface->surface_ops->surface_draw_overlay(surface); + else + return WINED3D_OK; + } /* Just overwrite the swapchain presentation interval. This is ok because * only ddraw apps can call Flip, and only d3d8 and d3d9 applications diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index ac21a89..40e7345 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1997,7 +1997,6 @@ struct wined3d_surface_ops void (*surface_map)(struct wined3d_surface *surface, const RECT *rect, DWORD flags); void (*surface_unmap)(struct wined3d_surface *surface); HRESULT (*surface_getdc)(struct wined3d_surface *surface); - HRESULT (*surface_flip)(struct wined3d_surface *surface, struct wined3d_surface *override); }; struct wined3d_surface

1 0

Henri Verbeet : wined3d: Make the device parameter to wined3d_device_get_transform() const.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: 82150458b470621bec8a52395d3bae0c4900f79b URL: http://source.winehq.org/git/wine.git/?a=commit;h=82150458b470621bec8a52395… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Aug 31 20:40:44 2011 +0200 wined3d: Make the device parameter to wined3d_device_get_transform() const. --- dlls/wined3d/device.c | 2 +- include/wine/wined3d.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index ba68866..ed8afc8 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -1899,7 +1899,7 @@ HRESULT CDECL wined3d_device_set_transform(struct wined3d_device *device, } -HRESULT CDECL wined3d_device_get_transform(struct wined3d_device *device, +HRESULT CDECL wined3d_device_get_transform(const struct wined3d_device *device, WINED3DTRANSFORMSTATETYPE state, WINED3DMATRIX *matrix) { TRACE("device %p, state %s, matrix %p.\n", device, debug_d3dtstype(state), matrix); diff --git a/include/wine/wined3d.h b/include/wine/wined3d.h index a038260..a5342bc 100644 --- a/include/wine/wined3d.h +++ b/include/wine/wined3d.h @@ -2259,7 +2259,7 @@ HRESULT __cdecl wined3d_device_get_texture(struct wined3d_device *device, UINT stage, struct wined3d_texture **texture); HRESULT __cdecl wined3d_device_get_texture_stage_state(struct wined3d_device *device, UINT stage, WINED3DTEXTURESTAGESTATETYPE state, DWORD *value); -HRESULT __cdecl wined3d_device_get_transform(struct wined3d_device *device, +HRESULT __cdecl wined3d_device_get_transform(const struct wined3d_device *device, WINED3DTRANSFORMSTATETYPE state, WINED3DMATRIX *matrix); HRESULT __cdecl wined3d_device_get_vertex_declaration(struct wined3d_device *device, struct wined3d_vertex_declaration **declaration);

1 0

Henri Verbeet : wined3d: Make the device parameter to wined3d_device_get_light() const.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: 9d9baef789361754b0dfc40ac75b6e3c133dc477 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9d9baef789361754b0dfc40ac… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Wed Aug 31 20:40:45 2011 +0200 wined3d: Make the device parameter to wined3d_device_get_light() const. --- dlls/wined3d/device.c | 2 +- include/wine/wined3d.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index ed8afc8..c64ac9e 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -2091,7 +2091,7 @@ HRESULT CDECL wined3d_device_set_light(struct wined3d_device *device, UINT light return WINED3D_OK; } -HRESULT CDECL wined3d_device_get_light(struct wined3d_device *device, UINT light_idx, WINED3DLIGHT *light) +HRESULT CDECL wined3d_device_get_light(const struct wined3d_device *device, UINT light_idx, WINED3DLIGHT *light) { UINT hash_idx = LIGHTMAP_HASHFUNC(light_idx); struct wined3d_light_info *light_info = NULL; diff --git a/include/wine/wined3d.h b/include/wine/wined3d.h index a5342bc..7b9a576 100644 --- a/include/wine/wined3d.h +++ b/include/wine/wined3d.h @@ -2222,7 +2222,7 @@ HRESULT __cdecl wined3d_device_get_front_buffer_data(struct wined3d_device *devi UINT swapchain_idx, struct wined3d_surface *dst_surface); void __cdecl wined3d_device_get_gamma_ramp(struct wined3d_device *device, UINT swapchain_idx, WINED3DGAMMARAMP *ramp); HRESULT __cdecl wined3d_device_get_index_buffer(struct wined3d_device *device, struct wined3d_buffer **index_buffer); -HRESULT __cdecl wined3d_device_get_light(struct wined3d_device *device, UINT light_idx, WINED3DLIGHT *light); +HRESULT __cdecl wined3d_device_get_light(const struct wined3d_device *device, UINT light_idx, WINED3DLIGHT *light); HRESULT __cdecl wined3d_device_get_light_enable(struct wined3d_device *device, UINT light_idx, BOOL *enable); HRESULT __cdecl wined3d_device_get_material(struct wined3d_device *device, WINED3DMATERIAL *material); float __cdecl wined3d_device_get_npatch_mode(struct wined3d_device *device);

1 0

Andrew Eikum : winmm: Determine the driver from MMDevAPI.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: f6890ef0e227afd12ce2325497d0cae478602c7d URL: http://source.winehq.org/git/wine.git/?a=commit;h=f6890ef0e227afd12ce232549… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Wed Aug 31 15:04:23 2011 -0500 winmm: Determine the driver from MMDevAPI. This uses a half-implemented, Wine-specific IMMDevice object to extract the driver name. --- dlls/mmdevapi/devenum.c | 95 +++++++++++++++++++++++++++++++++++++++++++++- dlls/mmdevapi/main.c | 1 + dlls/mmdevapi/mmdevapi.h | 1 + dlls/winmm/lolvldrv.c | 80 +++++++++++++++++++++++++++++--------- 4 files changed, 157 insertions(+), 20 deletions(-) diff --git a/dlls/mmdevapi/devenum.c b/dlls/mmdevapi/devenum.c index 8a71262..0cb471b 100644 --- a/dlls/mmdevapi/devenum.c +++ b/dlls/mmdevapi/devenum.c @@ -85,6 +85,8 @@ static const IMMDeviceVtbl MMDeviceVtbl; static const IPropertyStoreVtbl MMDevPropVtbl; static const IMMEndpointVtbl MMEndpointVtbl; +static IMMDevice info_device; + typedef struct MMDevColImpl { IMMDeviceCollection IMMDeviceCollection_iface; @@ -935,10 +937,19 @@ static HRESULT WINAPI MMDevEnum_GetDevice(IMMDeviceEnumerator *iface, const WCHA DWORD i=0; IMMDevice *dev = NULL; + static const WCHAR wine_info_deviceW[] = {'W','i','n','e',' ', + 'i','n','f','o',' ','d','e','v','i','c','e',0}; + + TRACE("(%p)->(%s,%p)\n", This, debugstr_w(name), device); + if(!name || !device) return E_POINTER; - TRACE("(%p)->(%s,%p)\n", This, debugstr_w(name), device); + if(!lstrcmpW(name, wine_info_deviceW)){ + *device = &info_device; + return S_OK; + } + for (i = 0; i < MMDevice_count; ++i) { WCHAR *str; @@ -1211,3 +1222,85 @@ static const IPropertyBagVtbl PB_Vtbl = PB_Read, PB_Write }; + +static ULONG WINAPI info_device_ps_AddRef(IPropertyStore *iface) +{ + return 2; +} + +static ULONG WINAPI info_device_ps_Release(IPropertyStore *iface) +{ + return 1; +} + +static HRESULT WINAPI info_device_ps_GetValue(IPropertyStore *iface, + REFPROPERTYKEY key, PROPVARIANT *pv) +{ + MMDevPropStore *This = impl_from_IPropertyStore(iface); + + if (!key || !pv) + return E_POINTER; + if (This->access != STGM_READ + && This->access != STGM_READWRITE) + return STG_E_ACCESSDENIED; + + if (IsEqualPropertyKey(*key, DEVPKEY_Device_Driver)) + { + pv->vt = VT_LPWSTR; + pv->u.pwszVal = CoTaskMemAlloc(lstrlenW(drvs.module_name) * sizeof(WCHAR)); + if (!pv->u.pwszVal) + return E_OUTOFMEMORY; + lstrcpyW(pv->u.pwszVal, drvs.module_name); + return S_OK; + } + + return E_INVALIDARG; +} + +static const IPropertyStoreVtbl info_device_ps_Vtbl = +{ + NULL, + info_device_ps_AddRef, + info_device_ps_Release, + NULL, + NULL, + info_device_ps_GetValue, + NULL, + NULL +}; + +static IPropertyStore info_device_ps = { + &info_device_ps_Vtbl +}; + +static ULONG WINAPI info_device_AddRef(IMMDevice *iface) +{ + return 2; +} + +static ULONG WINAPI info_device_Release(IMMDevice *iface) +{ + return 1; +} + +static HRESULT WINAPI info_device_OpenPropertyStore(IMMDevice *iface, + DWORD access, IPropertyStore **ppv) +{ + *ppv = &info_device_ps; + return S_OK; +} + +static const IMMDeviceVtbl info_device_Vtbl = +{ + NULL, + info_device_AddRef, + info_device_Release, + NULL, + info_device_OpenPropertyStore, + NULL, + NULL +}; + +static IMMDevice info_device = { + &info_device_Vtbl +}; diff --git a/dlls/mmdevapi/main.c b/dlls/mmdevapi/main.c index c694c6e..57541ab 100644 --- a/dlls/mmdevapi/main.c +++ b/dlls/mmdevapi/main.c @@ -79,6 +79,7 @@ static BOOL load_driver(const WCHAR *name) LDFC(GetAudioSessionManager); #undef LDFC + lstrcpyW(drvs.module_name, driver_module); TRACE("Successfully loaded %s\n", wine_dbgstr_w(driver_module)); return TRUE; diff --git a/dlls/mmdevapi/mmdevapi.h b/dlls/mmdevapi/mmdevapi.h index 2c3cd2d..e61c539 100644 --- a/dlls/mmdevapi/mmdevapi.h +++ b/dlls/mmdevapi/mmdevapi.h @@ -27,6 +27,7 @@ extern HRESULT MMDevice_GetPropValue(const GUID *devguid, DWORD flow, REFPROPERT typedef struct _DriverFuncs { HMODULE module; + WCHAR module_name[64]; /* ids gets an array of human-friendly endpoint names * keys gets an array of driver-specific stuff that is used diff --git a/dlls/winmm/lolvldrv.c b/dlls/winmm/lolvldrv.c index 907d356..b8f1b54 100644 --- a/dlls/winmm/lolvldrv.c +++ b/dlls/winmm/lolvldrv.c @@ -23,6 +23,9 @@ #include "config.h" #include "wine/port.h" +#define NONAMELESSUNION +#define NONAMELESSSTRUCT +#define COBJMACROS #include <string.h> #include <stdarg.h> #include <stdio.h> @@ -30,10 +33,16 @@ #include "windef.h" #include "winbase.h" #include "winreg.h" +#include "winnls.h" #include "winemm.h" #include "wine/debug.h" #include "wine/exception.h" +#include "wingdi.h" +#include "ole2.h" +#include "devpkey.h" +#include "mmdeviceapi.h" + WINE_DEFAULT_DEBUG_CHANNEL(winmm); /* Default set of drivers to be loaded */ @@ -509,34 +518,67 @@ static BOOL MMDRV_Install(LPCSTR drvRegName, LPCSTR drvFileName, BOOL bIsMapper) */ static void MMDRV_Init(void) { - HKEY hKey; - char driver_buffer[256]; - char *p, *next; + IMMDeviceEnumerator *devenum; + IMMDevice *device; + IPropertyStore *ps; + PROPVARIANT pv; + DWORD size; + char *drvA; + HRESULT init_hr, hr; + + static const WCHAR wine_info_deviceW[] = {'W','i','n','e',' ', + 'i','n','f','o',' ','d','e','v','i','c','e',0}; + TRACE("()\n"); - strcpy(driver_buffer, WINE_DEFAULT_WINMM_DRIVER); + init_hr = CoInitializeEx(NULL, COINIT_APARTMENTTHREADED); - /* @@ Wine registry key: HKCU\Software\Wine\Drivers */ - if (!RegOpenKeyA(HKEY_CURRENT_USER, "Software\\Wine\\Drivers", &hKey)) - { - DWORD size = sizeof(driver_buffer); - if (RegQueryValueExA(hKey, "Audio", 0, NULL, (BYTE*)driver_buffer, &size)) - strcpy(driver_buffer, WINE_DEFAULT_WINMM_DRIVER); + hr = CoCreateInstance(&CLSID_MMDeviceEnumerator, NULL, + CLSCTX_INPROC_SERVER, &IID_IMMDeviceEnumerator, (void**)&devenum); + if(FAILED(hr)){ + ERR("CoCreateInstance failed: %08x\n", hr); + goto exit; } - for (p = driver_buffer; p; p = next) - { - char filename[sizeof(driver_buffer)+10]; - next = strchr(p, ','); - if (next) *next++ = 0; - sprintf( filename, "wine%s.drv", p ); - if (MMDRV_Install(filename, filename, FALSE)) - break; - p = next; + hr = IMMDeviceEnumerator_GetDevice(devenum, wine_info_deviceW, &device); + IMMDeviceEnumerator_Release(devenum); + if(FAILED(hr)){ + ERR("GetDevice failed: %08x\n", hr); + goto exit; + } + + hr = IMMDevice_OpenPropertyStore(device, STGM_READ, &ps); + if(FAILED(hr)){ + ERR("OpenPropertyStore failed: %08x\n", hr); + IMMDevice_Release(device); + goto exit; } + hr = IPropertyStore_GetValue(ps, + (const PROPERTYKEY *)&DEVPKEY_Device_Driver, &pv); + IPropertyStore_Release(ps); + IMMDevice_Release(device); + if(FAILED(hr)){ + ERR("GetValue failed: %08x\n", hr); + goto exit; + } + + size = WideCharToMultiByte(CP_ACP, 0, pv.u.pwszVal, -1, + NULL, 0, NULL, NULL); + drvA = HeapAlloc(GetProcessHeap(), 0, size); + WideCharToMultiByte(CP_ACP, 0, pv.u.pwszVal, -1, drvA, size, NULL, NULL); + + MMDRV_Install(drvA, drvA, FALSE); + + HeapFree(GetProcessHeap(), 0, drvA); + PropVariantClear(&pv); + MMDRV_Install("wavemapper", "msacm32.drv", TRUE); MMDRV_Install("midimapper", "midimap.dll", TRUE); + +exit: + if(SUCCEEDED(init_hr)) + CoUninitialize(); } /******************************************************************

1 0

Andrew Eikum : mmdevapi: Automatically select the correct driver.
by Alexandre Julliard 01 Sep '11

01 Sep '11

Module: wine Branch: master Commit: c4b94b1ba4ee1664c0a14515d6fbc6d648e9175b URL: http://source.winehq.org/git/wine.git/?a=commit;h=c4b94b1ba4ee1664c0a14515d… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Wed Aug 31 15:04:37 2011 -0500 mmdevapi: Automatically select the correct driver. --- dlls/mmdevapi/main.c | 97 ++++++++++++++++--------- dlls/mmdevapi/mmdevapi.h | 15 ++++ dlls/winealsa.drv/mmdevdrv.c | 13 ++++ dlls/winealsa.drv/winealsa.drv.spec | 1 + dlls/winecoreaudio.drv/mmdevdrv.c | 13 ++++ dlls/winecoreaudio.drv/winecoreaudio.drv.spec | 1 + dlls/wineoss.drv/mmdevdrv.c | 48 ++++++++++++ dlls/wineoss.drv/wineoss.drv.spec | 1 + 8 files changed, 155 insertions(+), 34 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=c4b94b1ba4ee1664c0a14…

1 0