wine-commits September 2014

wine-commits@winehq.org

2 participants
301 discussions

Andreas Fuchs : kernel32: Ignore flags on FindFirstFileExW.
by Alexandre Julliard 29 Sep '14

29 Sep '14

Module: wine Branch: master Commit: ddeab7e564fc302cffa5a3b17134a1fb2f3f26b4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ddeab7e564fc302cffa5a3b17… Author: Andreas Fuchs <anduchs(a)gmail.com> Date: Mon Sep 29 16:37:41 2014 +0200 kernel32: Ignore flags on FindFirstFileExW. --- dlls/kernel32/file.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/dlls/kernel32/file.c b/dlls/kernel32/file.c index cf683df..512b63e 100644 --- a/dlls/kernel32/file.c +++ b/dlls/kernel32/file.c @@ -1865,10 +1865,13 @@ HANDLE WINAPI FindFirstFileExW( LPCWSTR filename, FINDEX_INFO_LEVELS level, TRACE("%s %d %p %d %p %x\n", debugstr_w(filename), level, data, search_op, filter, flags); - if ((search_op != FindExSearchNameMatch && search_op != FindExSearchLimitToDirectories) - || flags != 0) + if (flags != 0) { - FIXME("options not implemented 0x%08x 0x%08x\n", search_op, flags ); + FIXME("flags not implemented 0x%08x\n", flags ); + } + if (search_op != FindExSearchNameMatch && search_op != FindExSearchLimitToDirectories) + { + FIXME("search_op not implemented 0x%08x\n", search_op); return INVALID_HANDLE_VALUE; } if (level != FindExInfoStandard)

1 0

Akihiro Sagawa : ntdll/tests: Add KeyNameInformation tests.
by Alexandre Julliard 29 Sep '14

29 Sep '14

Module: wine Branch: master Commit: e47ff906597bf38dbc439da8e4d48b59c2334655 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e47ff906597bf38dbc439da8e… Author: Akihiro Sagawa <sagawa.aki(a)gmail.com> Date: Tue Sep 30 00:13:50 2014 +0900 ntdll/tests: Add KeyNameInformation tests. --- dlls/ntdll/tests/reg.c | 41 +++++++++++++++++++++++++++++++++++++++++ 1 file changed, 41 insertions(+) diff --git a/dlls/ntdll/tests/reg.c b/dlls/ntdll/tests/reg.c index db608af..22048f9 100644 --- a/dlls/ntdll/tests/reg.c +++ b/dlls/ntdll/tests/reg.c @@ -129,11 +129,13 @@ static NTSTATUS (WINAPI * pNtDeleteKey)(HANDLE); static NTSTATUS (WINAPI * pNtCreateKey)( PHANDLE retkey, ACCESS_MASK access, const OBJECT_ATTRIBUTES *attr, ULONG TitleIndex, const UNICODE_STRING *class, ULONG options, PULONG dispos ); +static NTSTATUS (WINAPI * pNtQueryKey)(HANDLE,KEY_INFORMATION_CLASS,PVOID,ULONG,PULONG); static NTSTATUS (WINAPI * pNtQueryValueKey)(HANDLE,const UNICODE_STRING *,KEY_VALUE_INFORMATION_CLASS,void *,DWORD,DWORD *); static NTSTATUS (WINAPI * pNtSetValueKey)(HANDLE, const PUNICODE_STRING, ULONG, ULONG, const void*, ULONG ); static NTSTATUS (WINAPI * pNtQueryInformationProcess)(HANDLE,PROCESSINFOCLASS,PVOID,ULONG,PULONG); static NTSTATUS (WINAPI * pRtlFormatCurrentUserKeyPath)(PUNICODE_STRING); +static LONG (WINAPI * pRtlCompareUnicodeString)(const PUNICODE_STRING,const PUNICODE_STRING,BOOLEAN); static BOOLEAN (WINAPI * pRtlCreateUnicodeString)(PUNICODE_STRING, LPCWSTR); static LPVOID (WINAPI * pRtlReAllocateHeap)(IN PVOID, IN ULONG, IN PVOID, IN ULONG); static NTSTATUS (WINAPI * pRtlAppendUnicodeToString)(PUNICODE_STRING, PCWSTR); @@ -174,11 +176,13 @@ static BOOL InitFunctionPtrs(void) NTDLL_GET_PROC(NtCreateKey) NTDLL_GET_PROC(NtFlushKey) NTDLL_GET_PROC(NtDeleteKey) + NTDLL_GET_PROC(NtQueryKey) NTDLL_GET_PROC(NtQueryValueKey) NTDLL_GET_PROC(NtQueryInformationProcess) NTDLL_GET_PROC(NtSetValueKey) NTDLL_GET_PROC(NtOpenKey) NTDLL_GET_PROC(RtlFormatCurrentUserKeyPath) + NTDLL_GET_PROC(RtlCompareUnicodeString) NTDLL_GET_PROC(RtlReAllocateHeap) NTDLL_GET_PROC(RtlAppendUnicodeToString) NTDLL_GET_PROC(RtlUnicodeStringToAnsiString) @@ -1276,6 +1280,42 @@ static void test_long_value_name(void) pNtClose(key); } +static void test_NtQueryKey(void) +{ + HANDLE key; + NTSTATUS status; + OBJECT_ATTRIBUTES attr; + ULONG len; + KEY_NAME_INFORMATION *info = NULL; + UNICODE_STRING str; + + InitializeObjectAttributes(&attr, &winetestpath, 0, 0, 0); + status = pNtOpenKey(&key, KEY_READ, &attr); + ok(status == STATUS_SUCCESS, "NtOpenKey Failed: 0x%08x\n", status); + + status = pNtQueryKey(key, KeyNameInformation, NULL, 0, &len); + if (status == STATUS_INVALID_PARAMETER) { + win_skip("KeyNameInformation is not supported\n"); + pNtClose(key); + return; + } + todo_wine ok(status == STATUS_BUFFER_TOO_SMALL, "NtQueryKey Failed: 0x%08x\n", status); + + info = HeapAlloc(GetProcessHeap(), 0, len); + status = pNtQueryKey(key, KeyNameInformation, info, len, &len); + ok(status == STATUS_SUCCESS, "NtQueryKey Failed: 0x%08x\n", status); + + str.Buffer = info->Name; + str.Length = info->NameLength; + todo_wine ok(pRtlCompareUnicodeString(&winetestpath, &str, TRUE) == 0, + "got %s, expected %s\n", + wine_dbgstr_wn(str.Buffer, str.Length/sizeof(WCHAR)), + wine_dbgstr_wn(winetestpath.Buffer, winetestpath.Length/sizeof(WCHAR))); + + HeapFree(GetProcessHeap(), 0, info); + pNtClose(key); +} + START_TEST(reg) { static const WCHAR winetest[] = {'\\','W','i','n','e','T','e','s','t',0}; @@ -1296,6 +1336,7 @@ START_TEST(reg) test_RtlQueryRegistryValues(); test_RtlpNtQueryValueKey(); test_NtFlushKey(); + test_NtQueryKey(); test_NtQueryValueKey(); test_long_value_name(); test_NtDeleteKey();

1 0

Akihiro Sagawa : ntdll/tests: Remove duplicated entry.
by Alexandre Julliard 29 Sep '14

29 Sep '14

Module: wine Branch: master Commit: 2be3df3c32c799b29c8a0e0c89f246ba272ca165 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2be3df3c32c799b29c8a0e0c8… Author: Akihiro Sagawa <sagawa.aki(a)gmail.com> Date: Tue Sep 30 00:13:48 2014 +0900 ntdll/tests: Remove duplicated entry. --- dlls/ntdll/tests/reg.c | 1 - 1 file changed, 1 deletion(-) diff --git a/dlls/ntdll/tests/reg.c b/dlls/ntdll/tests/reg.c index 1594665..db608af 100644 --- a/dlls/ntdll/tests/reg.c +++ b/dlls/ntdll/tests/reg.c @@ -166,7 +166,6 @@ static BOOL InitFunctionPtrs(void) NTDLL_GET_PROC(RtlCreateUnicodeStringFromAsciiz) NTDLL_GET_PROC(RtlCreateUnicodeString) NTDLL_GET_PROC(RtlFreeUnicodeString) - NTDLL_GET_PROC(NtDeleteValueKey) NTDLL_GET_PROC(RtlQueryRegistryValues) NTDLL_GET_PROC(RtlCheckRegistryKey) NTDLL_GET_PROC(RtlOpenCurrentUser)

1 0

Henri Verbeet : wined3d: Get rid of SFLAG_DONOTFREE.
by Alexandre Julliard 29 Sep '14

29 Sep '14

Module: wine Branch: master Commit: 2d5c8aff3ccbbdcf3a8c9d6e3262d894684d1a9c URL: http://source.winehq.org/git/wine.git/?a=commit;h=2d5c8aff3ccbbdcf3a8c9d6e3… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Sep 29 09:11:55 2014 +0200 wined3d: Get rid of SFLAG_DONOTFREE. --- dlls/wined3d/surface.c | 7 ++++++- dlls/wined3d/wined3d_private.h | 10 ---------- 2 files changed, 6 insertions(+), 11 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index add864a..955c1b6 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -587,7 +587,12 @@ void surface_prepare_map_memory(struct wined3d_surface *surface) static void surface_evict_sysmem(struct wined3d_surface *surface) { - if (surface->resource.map_count || surface->flags & SFLAG_DONOTFREE) + /* In some conditions the surface memory must not be freed: + * SFLAG_CONVERTED: Converting the data back would take too long + * SFLAG_DYNLOCK: Avoid freeing the data for performance + * SFLAG_CLIENT: OpenGL uses our memory as backup */ + if (surface->resource.map_count || surface->flags & (SFLAG_CONVERTED | SFLAG_DYNLOCK + | SFLAG_CLIENT | SFLAG_PIN_SYSMEM)) return; wined3d_resource_free_sysmem(&surface->resource); diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 76ac66a..110334b 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -2350,16 +2350,6 @@ void flip_surface(struct wined3d_surface *front, struct wined3d_surface *back) D #define SFLAG_ALLOCATED 0x00000800 /* A GL texture is allocated for this surface. */ #define SFLAG_SRGBALLOCATED 0x00001000 /* A sRGB GL texture is allocated for this surface. */ -/* In some conditions the surface memory must not be freed: - * SFLAG_CONVERTED: Converting the data back would take too long - * SFLAG_DYNLOCK: Avoid freeing the data for performance - * SFLAG_CLIENT: OpenGL uses our memory as backup - */ -#define SFLAG_DONOTFREE (SFLAG_CONVERTED | \ - SFLAG_DYNLOCK | \ - SFLAG_CLIENT | \ - SFLAG_PIN_SYSMEM) - enum wined3d_conversion_type { WINED3D_CT_NONE,

1 0

Henri Verbeet : wined3d: Replace wined3d_surface_update_desc() with wined3d_texture_update_desc().
by Alexandre Julliard 29 Sep '14

29 Sep '14

Module: wine Branch: master Commit: 97856bcfee78122060e2caf80ad16804074401d5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=97856bcfee78122060e2caf80… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Sep 29 09:11:54 2014 +0200 wined3d: Replace wined3d_surface_update_desc() with wined3d_texture_update_desc(). --- dlls/d3d9/device.c | 14 ++++-------- dlls/ddraw/surface.c | 4 ++-- dlls/wined3d/device.c | 6 ++--- dlls/wined3d/surface.c | 52 +++++++++++------------------------------- dlls/wined3d/texture.c | 48 ++++++++++++++++++++++++++++++++++++++ dlls/wined3d/wined3d.spec | 2 +- dlls/wined3d/wined3d_private.h | 2 ++ include/wine/wined3d.h | 8 +++---- 8 files changed, 76 insertions(+), 60 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=97856bcfee78122060e2c…

1 0

Henri Verbeet : ddraw: Set user memory in ddraw_surface_create() instead of ddraw_surface_init().
by Alexandre Julliard 29 Sep '14

29 Sep '14

Module: wine Branch: master Commit: 5000eba6d094a39f82c6814e829c0a03f2ebaf04 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5000eba6d094a39f82c6814e8… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Sep 29 09:11:53 2014 +0200 ddraw: Set user memory in ddraw_surface_create() instead of ddraw_surface_init(). --- dlls/ddraw/ddraw.c | 9 +------- dlls/ddraw/ddraw_private.h | 2 +- dlls/ddraw/surface.c | 54 +++++++++++++++------------------------------- 3 files changed, 19 insertions(+), 46 deletions(-) diff --git a/dlls/ddraw/ddraw.c b/dlls/ddraw/ddraw.c index 18c30da..36eb679 100644 --- a/dlls/ddraw/ddraw.c +++ b/dlls/ddraw/ddraw.c @@ -4726,7 +4726,6 @@ static HRESULT CDECL device_parent_surface_created(struct wined3d_device_parent { struct ddraw *ddraw = ddraw_from_device_parent(device_parent); struct ddraw_surface *ddraw_surface; - HRESULT hr; TRACE("device_parent %p, container_parent %p, surface %p, parent %p, parent_ops %p.\n", device_parent, container_parent, surface, parent, parent_ops); @@ -4746,13 +4745,7 @@ static HRESULT CDECL device_parent_surface_created(struct wined3d_device_parent return DDERR_OUTOFVIDEOMEMORY; } - if (FAILED(hr = ddraw_surface_init(ddraw_surface, ddraw, container_parent, surface, parent_ops))) - { - WARN("Failed to initialize surface, hr %#x.\n", hr); - HeapFree(GetProcessHeap(), 0, ddraw_surface); - return hr; - } - + ddraw_surface_init(ddraw_surface, ddraw, container_parent, surface, parent_ops); *parent = ddraw_surface; list_add_head(&ddraw->surface_list, &ddraw_surface->surface_list_entry); diff --git a/dlls/ddraw/ddraw_private.h b/dlls/ddraw/ddraw_private.h index 7d028dd..a2eee1a 100644 --- a/dlls/ddraw/ddraw_private.h +++ b/dlls/ddraw/ddraw_private.h @@ -207,7 +207,7 @@ struct ddraw_texture HRESULT ddraw_surface_create(struct ddraw *ddraw, const DDSURFACEDESC2 *surface_desc, struct ddraw_surface **surface, IUnknown *outer_unknown, unsigned int version) DECLSPEC_HIDDEN; struct wined3d_rendertarget_view *ddraw_surface_get_rendertarget_view(struct ddraw_surface *surface) DECLSPEC_HIDDEN; -HRESULT ddraw_surface_init(struct ddraw_surface *surface, struct ddraw *ddraw, struct ddraw_texture *texture, +void ddraw_surface_init(struct ddraw_surface *surface, struct ddraw *ddraw, struct ddraw_texture *texture, struct wined3d_surface *wined3d_surface, const struct wined3d_parent_ops **parent_ops) DECLSPEC_HIDDEN; ULONG ddraw_surface_release_iface(struct ddraw_surface *This) DECLSPEC_HIDDEN; HRESULT ddraw_surface_update_frontbuffer(struct ddraw_surface *surface, diff --git a/dlls/ddraw/surface.c b/dlls/ddraw/surface.c index 0006126..8c48140 100644 --- a/dlls/ddraw/surface.c +++ b/dlls/ddraw/surface.c @@ -5605,6 +5605,7 @@ HRESULT ddraw_surface_create(struct ddraw *ddraw, const DDSURFACEDESC2 *surface_ DDSURFACEDESC2 *desc, *mip_desc; struct ddraw_texture *texture; UINT layers, levels, i, j; + unsigned int pitch = 0; HRESULT hr; TRACE("ddraw %p, surface_desc %p, surface %p, outer_unknown %p, version %u.\n", @@ -5969,6 +5970,8 @@ HRESULT ddraw_surface_create(struct ddraw *ddraw, const DDSURFACEDESC2 *surface_ HeapFree(GetProcessHeap(), 0, texture); return DDERR_INVALIDPARAMS; } + + pitch = desc->u1.lPitch; } } @@ -6073,6 +6076,14 @@ HRESULT ddraw_surface_create(struct ddraw *ddraw, const DDSURFACEDESC2 *surface_ } } + if ((desc->dwFlags & DDSD_LPSURFACE) && FAILED(hr = wined3d_surface_update_desc(root->wined3d_surface, + wined3d_desc.width, wined3d_desc.height, wined3d_desc.format, + WINED3D_MULTISAMPLE_NONE, 0, desc->lpSurface, pitch))) + { + ERR("Failed to set surface memory, hr %#x.\n", hr); + goto fail; + } + if (desc->dwFlags & DDSD_BACKBUFFERCOUNT) { unsigned int count = desc->dwBackBufferCount; @@ -6148,13 +6159,12 @@ fail: return hr; } -HRESULT ddraw_surface_init(struct ddraw_surface *surface, struct ddraw *ddraw, struct ddraw_texture *texture, +void ddraw_surface_init(struct ddraw_surface *surface, struct ddraw *ddraw, struct ddraw_texture *texture, struct wined3d_surface *wined3d_surface, const struct wined3d_parent_ops **parent_ops) { DDSURFACEDESC2 *desc = &surface->surface_desc; struct wined3d_resource_desc wined3d_desc; unsigned int version = texture->version; - HRESULT hr; surface->IDirectDrawSurface7_iface.lpVtbl = &ddraw_surface7_vtbl; surface->IDirectDrawSurface4_iface.lpVtbl = &ddraw_surface4_vtbl; @@ -6193,46 +6203,18 @@ HRESULT ddraw_surface_init(struct ddraw_surface *surface, struct ddraw *ddraw, s if (format_is_compressed(&desc->u4.ddpfPixelFormat)) { if (desc->dwFlags & DDSD_LPSURFACE) - { - if (FAILED(hr = wined3d_surface_update_desc(wined3d_surface, wined3d_desc.width, - wined3d_desc.height, wined3d_desc.format, WINED3D_MULTISAMPLE_NONE, 0, - desc->lpSurface, 0))) - { - ERR("Failed to set surface memory, hr %#x.\n", hr); - return hr; - } - - desc->dwFlags |= DDSD_LINEARSIZE; - desc->dwFlags &= ~(DDSD_LPSURFACE | DDSD_PITCH); desc->u1.dwLinearSize = ~0u; - } else - { - desc->dwFlags |= DDSD_LINEARSIZE; - desc->dwFlags &= ~DDSD_PITCH; desc->u1.dwLinearSize = wined3d_surface_get_pitch(wined3d_surface) * ((desc->dwHeight + 3) / 4); - } + desc->dwFlags |= DDSD_LINEARSIZE; + desc->dwFlags &= ~(DDSD_LPSURFACE | DDSD_PITCH); } else { - if (desc->dwFlags & DDSD_LPSURFACE) - { - if (FAILED(hr = wined3d_surface_update_desc(wined3d_surface, wined3d_desc.width, - wined3d_desc.height, wined3d_desc.format, WINED3D_MULTISAMPLE_NONE, 0, - desc->lpSurface, desc->u1.lPitch))) - { - ERR("Failed to set surface memory, hr %#x.\n", hr); - return hr; - } - - desc->dwFlags &= ~(DDSD_LPSURFACE | DDSD_LINEARSIZE); - } - else - { - desc->dwFlags |= DDSD_PITCH; - desc->dwFlags &= ~DDSD_LINEARSIZE; + if (!(desc->dwFlags & DDSD_LPSURFACE)) desc->u1.lPitch = wined3d_surface_get_pitch(wined3d_surface); - } + desc->dwFlags |= DDSD_PITCH; + desc->dwFlags &= ~(DDSD_LPSURFACE | DDSD_LINEARSIZE); } desc->lpSurface = NULL; @@ -6241,8 +6223,6 @@ HRESULT ddraw_surface_init(struct ddraw_surface *surface, struct ddraw *ddraw, s *parent_ops = &ddraw_surface_wined3d_parent_ops; wined3d_private_store_init(&surface->private_store); - - return DD_OK; } static void STDMETHODCALLTYPE view_wined3d_object_destroyed(void *parent)

1 0

Henri Verbeet : ddraw: Move the pitch / linear size validation from ddraw_surface_init() to ddraw_surface_create( ).
by Alexandre Julliard 29 Sep '14

29 Sep '14

Module: wine Branch: master Commit: e28800c6a41b766a61c94001c593753bcf59e519 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e28800c6a41b766a61c94001c… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Sep 29 09:11:52 2014 +0200 ddraw: Move the pitch / linear size validation from ddraw_surface_init() to ddraw_surface_create(). --- dlls/ddraw/surface.c | 31 +++++++++++++++++-------------- 1 file changed, 17 insertions(+), 14 deletions(-) diff --git a/dlls/ddraw/surface.c b/dlls/ddraw/surface.c index 0214a7f..0006126 100644 --- a/dlls/ddraw/surface.c +++ b/dlls/ddraw/surface.c @@ -5943,6 +5943,15 @@ HRESULT ddraw_surface_create(struct ddraw *ddraw, const DDSURFACEDESC2 *surface_ HeapFree(GetProcessHeap(), 0, texture); return DDERR_INVALIDPARAMS; } + + if ((desc->dwFlags & DDSD_LINEARSIZE) + && desc->u1.dwLinearSize < wined3d_calculate_format_pitch(ddraw->wined3d, WINED3DADAPTER_DEFAULT, + wined3d_desc.format, wined3d_desc.width) * ((desc->dwHeight + 3) / 4)) + { + WARN("Invalid linear size %u specified.\n", desc->u1.dwLinearSize); + HeapFree(GetProcessHeap(), 0, texture); + return DDERR_INVALIDPARAMS; + } } else { @@ -5952,6 +5961,14 @@ HRESULT ddraw_surface_create(struct ddraw *ddraw, const DDSURFACEDESC2 *surface_ HeapFree(GetProcessHeap(), 0, texture); return DDERR_INVALIDPARAMS; } + + if (desc->u1.lPitch < wined3d_calculate_format_pitch(ddraw->wined3d, WINED3DADAPTER_DEFAULT, + wined3d_desc.format, wined3d_desc.width) || desc->u1.lPitch & 3) + { + WARN("Invalid pitch %u specified.\n", desc->u1.lPitch); + HeapFree(GetProcessHeap(), 0, texture); + return DDERR_INVALIDPARAMS; + } } } @@ -6177,13 +6194,6 @@ HRESULT ddraw_surface_init(struct ddraw_surface *surface, struct ddraw *ddraw, s { if (desc->dwFlags & DDSD_LPSURFACE) { - if ((desc->dwFlags & DDSD_LINEARSIZE) - && desc->u1.dwLinearSize < wined3d_surface_get_pitch(wined3d_surface) * ((desc->dwHeight + 3) / 4)) - { - WARN("Invalid linear size %u specified.\n", desc->u1.dwLinearSize); - return DDERR_INVALIDPARAMS; - } - if (FAILED(hr = wined3d_surface_update_desc(wined3d_surface, wined3d_desc.width, wined3d_desc.height, wined3d_desc.format, WINED3D_MULTISAMPLE_NONE, 0, desc->lpSurface, 0))) @@ -6207,13 +6217,6 @@ HRESULT ddraw_surface_init(struct ddraw_surface *surface, struct ddraw *ddraw, s { if (desc->dwFlags & DDSD_LPSURFACE) { - if (desc->u1.lPitch < wined3d_calculate_format_pitch(ddraw->wined3d, WINED3DADAPTER_DEFAULT, - wined3d_desc.format, wined3d_desc.width) || desc->u1.lPitch & 3) - { - WARN("Invalid pitch %u specified.\n", desc->u1.lPitch); - return DDERR_INVALIDPARAMS; - } - if (FAILED(hr = wined3d_surface_update_desc(wined3d_surface, wined3d_desc.width, wined3d_desc.height, wined3d_desc.format, WINED3D_MULTISAMPLE_NONE, 0, desc->lpSurface, desc->u1.lPitch)))

1 0

Henri Verbeet : d3d8: Get rid of the format switching code in d3d8_device_CopyRects().
by Alexandre Julliard 29 Sep '14

29 Sep '14

Module: wine Branch: master Commit: d5365aae28255c8cc55f12d39cc7fec8428a8889 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d5365aae28255c8cc55f12d39… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Sep 29 09:11:51 2014 +0200 d3d8: Get rid of the format switching code in d3d8_device_CopyRects(). This can't possibly do anything, since we fail texture creation for WINED3DFMT_UNKNOWN. If texture creation with D3DFMT_UNKNOWN is actually supposed to work, we should probably pick an appropriate format at creation, instead of during CopyRects(). This code has existed in some form or another since the initial version in commit 850a9429dbc07c0b38e9f9f60aaec43759326d24. No tests, of course. --- dlls/d3d8/device.c | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git a/dlls/d3d8/device.c b/dlls/d3d8/device.c index 8ddb0cb..dd0b9ee 100644 --- a/dlls/d3d8/device.c +++ b/dlls/d3d8/device.c @@ -988,7 +988,6 @@ static HRESULT WINAPI d3d8_device_CopyRects(IDirect3DDevice8 *iface, struct wined3d_resource_desc wined3d_desc; struct wined3d_resource *wined3d_resource; UINT src_w, src_h; - HRESULT hr; TRACE("iface %p, src_surface %p, src_rects %p, rect_count %u, dst_surface %p, dst_points %p.\n", iface, src_surface, src_rects, rect_count, dst_surface, dst_points); @@ -1020,24 +1019,13 @@ static HRESULT WINAPI d3d8_device_CopyRects(IDirect3DDevice8 *iface, dst_format = wined3d_desc.format; /* Check that the source and destination formats match */ - if (src_format != dst_format && WINED3DFMT_UNKNOWN != dst_format) + if (src_format != dst_format) { WARN("Source %p format must match the destination %p format, returning D3DERR_INVALIDCALL.\n", src_surface, dst_surface); wined3d_mutex_unlock(); return D3DERR_INVALIDCALL; } - else if (WINED3DFMT_UNKNOWN == dst_format) - { - TRACE("Converting destination surface from WINED3DFMT_UNKNOWN to the source format.\n"); - if (FAILED(hr = wined3d_surface_update_desc(dst->wined3d_surface, wined3d_desc.width, wined3d_desc.height, - src_format, wined3d_desc.multisample_type, wined3d_desc.multisample_quality, NULL, 0))) - { - WARN("Failed to update surface desc, hr %#x.\n", hr); - wined3d_mutex_unlock(); - return hr; - } - } /* Quick if complete copy ... */ if (!rect_count && !src_rects && !dst_points)

1 0

Alistair Leslie-Hughes : dpnet: Implement IDirectPlay8Address Duplicate.
by Alexandre Julliard 29 Sep '14

29 Sep '14

Module: wine Branch: master Commit: 7dcfb9d3e55bab5a3422c85e85221e5279689d09 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7dcfb9d3e55bab5a3422c85e8… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Tue May 13 12:57:47 2014 +1000 dpnet: Implement IDirectPlay8Address Duplicate. --- dlls/dpnet/address.c | 33 ++++++++++++++++++++++++++++++--- dlls/dpnet/dpnet_private.h | 1 + dlls/dpnet/tests/address.c | 39 +++++++++++++++++++++++++++++++++++++++ 3 files changed, 70 insertions(+), 3 deletions(-) diff --git a/dlls/dpnet/address.c b/dlls/dpnet/address.c index 2cea1ea..7df7ad6 100644 --- a/dlls/dpnet/address.c +++ b/dlls/dpnet/address.c @@ -179,9 +179,35 @@ static HRESULT WINAPI IDirectPlay8AddressImpl_BuildFromURLA(IDirectPlay8Address static HRESULT WINAPI IDirectPlay8AddressImpl_Duplicate(IDirectPlay8Address *iface, IDirectPlay8Address **ppdpaNewAddress) { - IDirectPlay8AddressImpl *This = impl_from_IDirectPlay8Address(iface); - TRACE("(%p, %p): stub\n", This, ppdpaNewAddress); - return DPN_OK; + IDirectPlay8AddressImpl *This = impl_from_IDirectPlay8Address(iface); + IDirectPlay8Address *dup; + HRESULT hr; + + TRACE("(%p, %p)\n", This, ppdpaNewAddress); + + if(!ppdpaNewAddress) + return E_POINTER; + + hr = DPNET_CreateDirectPlay8Address(NULL, NULL, &IID_IDirectPlay8Address, (LPVOID*)&dup); + if(hr == S_OK) + { + IDirectPlay8AddressImpl *DupThis = impl_from_IDirectPlay8Address(dup); + struct component *entry; + + DupThis->SP_guid = This->SP_guid; + DupThis->init = This->init; + + LIST_FOR_EACH_ENTRY(entry, &This->components, struct component, entry) + { + hr = IDirectPlay8Address_AddComponent(dup, entry->name, &entry->data, entry->size, entry->type); + if(hr != S_OK) + ERR("Failed to copy component: %s - 0x%08x\n", debugstr_w(entry->name), hr); + } + + *ppdpaNewAddress = dup; + } + + return hr; } static HRESULT WINAPI IDirectPlay8AddressImpl_SetEqual(IDirectPlay8Address *iface, @@ -338,6 +364,7 @@ static HRESULT WINAPI IDirectPlay8AddressImpl_AddComponent(IDirectPlay8Address * list_add_tail(&This->components, &entry->entry); } + entry->size = dwDataSize; switch (dwDataType) { case DPNA_DATATYPE_DWORD: diff --git a/dlls/dpnet/dpnet_private.h b/dlls/dpnet/dpnet_private.h index 61e2200..1ed4f25 100644 --- a/dlls/dpnet/dpnet_private.h +++ b/dlls/dpnet/dpnet_private.h @@ -68,6 +68,7 @@ struct component WCHAR *name; DWORD type; + DWORD size; union { diff --git a/dlls/dpnet/tests/address.c b/dlls/dpnet/tests/address.c index 0f0b8ce..ad35954 100644 --- a/dlls/dpnet/tests/address.c +++ b/dlls/dpnet/tests/address.c @@ -214,6 +214,44 @@ static void address_setsp(void) } } +static void address_duplicate(void) +{ + HRESULT hr; + IDirectPlay8Address *localaddr = NULL; + IDirectPlay8Address *duplicate = NULL; + DWORD components, dupcomps; + GUID guid = IID_Random; + + hr = CoCreateInstance( &CLSID_DirectPlay8Address, NULL, CLSCTX_ALL, &IID_IDirectPlay8Address, (LPVOID*)&localaddr); + ok(hr == S_OK, "Failed to create IDirectPlay8Address object\n"); + if(SUCCEEDED(hr)) + { + hr = IDirectPlay8Address_SetSP(localaddr, &CLSID_DP8SP_TCPIP); + ok(hr == S_OK, "got 0x%08x\n", hr); + + hr = IDirectPlay8Address_GetNumComponents(localaddr, &components); + ok(hr == S_OK, "got 0x%08x\n", hr); + ok(components == 1, "components=%d\n", components); + + hr = IDirectPlay8Address_Duplicate(localaddr, &duplicate); + ok(hr == S_OK, "got 0x%08x\n", hr); + if(SUCCEEDED(hr)) + { + hr = IDirectPlay8Address_GetSP(duplicate, &guid); + ok(hr == S_OK, "got 0x%08x\n", hr); + ok(IsEqualGUID(&guid, &CLSID_DP8SP_TCPIP), "wrong guid\n"); + + hr = IDirectPlay8Address_GetNumComponents(duplicate, &dupcomps); + ok(hr == S_OK, "got 0x%08x\n", hr); + ok(components == dupcomps, "expected %d got %d\n", components, dupcomps); + + IDirectPlay8Address_Release(duplicate); + } + + IDirectPlay8Address_Release(localaddr); + } +} + START_TEST(address) { HRESULT hr; @@ -226,6 +264,7 @@ START_TEST(address) create_directplay_address(); address_addcomponents(); address_setsp(); + address_duplicate(); CoUninitialize(); }

1 0

Francois Gouget : Assorted spelling fixes.
by Alexandre Julliard 29 Sep '14

29 Sep '14

Module: wine Branch: master Commit: 0dd5cb1cef2d50653377e304ac76693792e48aae URL: http://source.winehq.org/git/wine.git/?a=commit;h=0dd5cb1cef2d50653377e304a… Author: Francois Gouget <fgouget(a)free.fr> Date: Sun Sep 28 23:53:46 2014 +0200 Assorted spelling fixes. --- dlls/comctl32/tests/button.c | 2 +- dlls/comdlg32/tests/itemdlg.c | 2 +- dlls/d3d8/tests/device.c | 2 +- dlls/ddraw/tests/ddraw1.c | 4 ++-- dlls/ddraw/tests/ddraw2.c | 4 ++-- dlls/ddraw/tests/ddraw4.c | 4 ++-- dlls/ddraw/tests/ddraw7.c | 4 ++-- dlls/dpvoice/tests/voice.c | 12 ++++++------ dlls/dwrite/analyzer.c | 2 +- dlls/kernel32/tests/pipe.c | 4 ++-- dlls/mshtml/tests/dom.c | 2 +- dlls/oleaut32/tests/vartype.c | 2 +- dlls/qcap/avimux.c | 2 +- dlls/rsaenh/tests/rsaenh.c | 2 +- dlls/shell32/tests/shlfileop.c | 2 +- dlls/wined3d/directx.c | 4 ++-- dlls/wmp/oleobj.c | 2 +- dlls/wmp/tests/oleobj.c | 2 +- dlls/xmllite/writer.c | 2 +- 19 files changed, 30 insertions(+), 30 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=0dd5cb1cef2d50653377e…

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits September 2014