September 2011 - Wine-commits

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

07 Sep '11

Module: wine Branch: master Commit: 33f087c95da2676ceb357c7cc6025538ae380284 URL: http://source.winehq.org/git/wine.git/?a=commit;h=33f087c95da2676ceb357c7cc… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Sep 6 19:19:52 2011 +0200 wined3d: Make the device parameter to wined3d_device_get_render_state() 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 16faace..4a8b5c6 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -2500,7 +2500,7 @@ HRESULT CDECL wined3d_device_set_render_state(struct wined3d_device *device, return WINED3D_OK; } -HRESULT CDECL wined3d_device_get_render_state(struct wined3d_device *device, +HRESULT CDECL wined3d_device_get_render_state(const struct wined3d_device *device, WINED3DRENDERSTATETYPE state, DWORD *value) { TRACE("device %p, state %s (%#x), value %p.\n", device, debug_d3drenderstate(state), state, value); diff --git a/include/wine/wined3d.h b/include/wine/wined3d.h index 23e88d1..2039398 100644 --- a/include/wine/wined3d.h +++ b/include/wine/wined3d.h @@ -2239,7 +2239,7 @@ HRESULT __cdecl wined3d_device_get_ps_consts_i(struct wined3d_device *device, UINT start_register, int *constants, UINT vector4i_count); HRESULT __cdecl wined3d_device_get_raster_status(struct wined3d_device *device, UINT swapchain_idx, WINED3DRASTER_STATUS *raster_status); -HRESULT __cdecl wined3d_device_get_render_state(struct wined3d_device *device, +HRESULT __cdecl wined3d_device_get_render_state(const struct wined3d_device *device, WINED3DRENDERSTATETYPE state, DWORD *value); HRESULT __cdecl wined3d_device_get_render_target(struct wined3d_device *device, UINT render_target_idx, struct wined3d_surface **render_target);

1 0

Stefan Dösinger : ddraw: Remove crashing tests.
by Alexandre Julliard 07 Sep '11

07 Sep '11

Module: wine Branch: master Commit: 252bead28027331c30887b331f55de2245d5d607 URL: http://source.winehq.org/git/wine.git/?a=commit;h=252bead28027331c30887b331… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Wed Sep 7 00:12:35 2011 +0200 ddraw: Remove crashing tests. --- dlls/ddraw/tests/ddrawmodes.c | 36 ------------------------------------ 1 files changed, 0 insertions(+), 36 deletions(-) diff --git a/dlls/ddraw/tests/ddrawmodes.c b/dlls/ddraw/tests/ddrawmodes.c index 61f07fe..81ff43e 100644 --- a/dlls/ddraw/tests/ddrawmodes.c +++ b/dlls/ddraw/tests/ddrawmodes.c @@ -104,13 +104,6 @@ static void releasedirectdraw(void) } } -static BOOL WINAPI crash_callbackA(GUID *lpGUID, LPSTR lpDriverDescription, - LPSTR lpDriverName, LPVOID lpContext) -{ - *(volatile char*)0 = 2; - return TRUE; -} - static BOOL WINAPI test_nullcontext_callbackA(GUID *lpGUID, LPSTR lpDriverDescription, LPSTR lpDriverName, LPVOID lpContext) { @@ -147,19 +140,6 @@ static void test_DirectDrawEnumerateA(void) ret = pDirectDrawEnumerateA(NULL, NULL); ok(ret == DDERR_INVALIDPARAMS, "Expected DDERR_INVALIDPARAMS, got %d\n", ret); - /* Test with invalid callback parameter. */ - ret = pDirectDrawEnumerateA((LPDDENUMCALLBACKA)0xdeadbeef, NULL); - ok(ret == DDERR_INVALIDPARAMS, "Expected DDERR_INVALIDPARAMS, got %d\n", ret); - - if (pDirectDrawEnumerateExA) - { - /* Test with callback that crashes. */ - ret = pDirectDrawEnumerateA(crash_callbackA, NULL); - ok(ret == DDERR_INVALIDPARAMS, "Expected DDERR_INVALIDPARAMS, got %d\n", ret); - } - else - win_skip("Test would crash on older ddraw\n"); - /* Test with valid callback parameter and NULL context parameter. */ trace("Calling DirectDrawEnumerateA with test_nullcontext_callbackA callback and NULL context.\n"); ret = pDirectDrawEnumerateA(test_nullcontext_callbackA, NULL); @@ -207,14 +187,6 @@ static void test_DirectDrawEnumerateW(void) ok(ret == DDERR_UNSUPPORTED, "Expected DDERR_UNSUPPORTED, got %d\n", ret); } -static BOOL WINAPI crash_callbackExA(GUID *lpGUID, LPSTR lpDriverDescription, - LPSTR lpDriverName, LPVOID lpContext, - HMONITOR hm) -{ - *(volatile char*)0 = 2; - return TRUE; -} - static BOOL WINAPI test_nullcontext_callbackExA(GUID *lpGUID, LPSTR lpDriverDescription, LPSTR lpDriverName, LPVOID lpContext, HMONITOR hm) @@ -253,14 +225,6 @@ static void test_DirectDrawEnumerateExA(void) ret = pDirectDrawEnumerateExA(NULL, NULL, 0); ok(ret == DDERR_INVALIDPARAMS, "Expected DDERR_INVALIDPARAMS, got %d\n", ret); - /* Test with invalid callback parameter. */ - ret = pDirectDrawEnumerateExA((LPDDENUMCALLBACKEXA)0xdeadbeef, NULL, 0); - ok(ret == DDERR_INVALIDPARAMS, "Expected DDERR_INVALIDPARAMS, got %d\n", ret); - - /* Test with callback that crashes. */ - ret = pDirectDrawEnumerateExA(crash_callbackExA, NULL, 0); - ok(ret == DDERR_INVALIDPARAMS, "Expected DDERR_INVALIDPARAMS, got %d\n", ret); - /* Test with valid callback parameter and invalid flags */ ret = pDirectDrawEnumerateExA(test_nullcontext_callbackExA, NULL, ~0); ok(ret == DDERR_INVALIDPARAMS, "Expected DDERR_INVALIDPARAMS, got %d\n", ret);

1 0

Stefan Dösinger : ddraw: Add some NULL pointer tests.
by Alexandre Julliard 07 Sep '11

07 Sep '11

Module: wine Branch: master Commit: 8e79a7e6644fd6f1e0cfeb8b826e683d171d723a URL: http://source.winehq.org/git/wine.git/?a=commit;h=8e79a7e6644fd6f1e0cfeb8b8… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sat Sep 3 21:06:11 2011 +0200 ddraw: Add some NULL pointer tests. --- dlls/ddraw/ddraw.c | 36 ++++++++++++------------ dlls/ddraw/surface.c | 26 +++++++++++------ dlls/ddraw/tests/ddrawmodes.c | 4 ++ dlls/ddraw/tests/dsurface.c | 62 +++++++++++++++++++++++++++++++++++++++++ 4 files changed, 101 insertions(+), 27 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=8e79a7e6644fd6f1e0cfe…

1 0

Stefan Dösinger : ddraw: Check the alignment, not the size.
by Alexandre Julliard 07 Sep '11

07 Sep '11

Module: wine Branch: master Commit: d6f17d686debb491d7213f973453659d6ffa76d8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d6f17d686debb491d7213f973… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sun Aug 28 17:18:09 2011 +0200 ddraw: Check the alignment, not the size. --- dlls/ddraw/tests/overlay.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/ddraw/tests/overlay.c b/dlls/ddraw/tests/overlay.c index 5053efb..9660d74 100644 --- a/dlls/ddraw/tests/overlay.c +++ b/dlls/ddraw/tests/overlay.c @@ -215,7 +215,7 @@ static void yv12_test(void) ok(desc.dwWidth == 256 && desc.dwHeight == 256, "Expected size 256x256, got %ux%u\n", desc.dwWidth, desc.dwHeight); /* The overlay pitch seems to have 256 byte alignment */ - ok(U1(desc).lPitch == 256, "Unexpected pitch %u, expected 256\n", U1(desc).lPitch); + ok((U1(desc).lPitch & 0xff) == 0, "Expected 256 byte aligned pitch, got %u\n", U1(desc).lPitch); hr = IDirectDrawSurface7_Unlock(surface, NULL); ok(hr == DD_OK, "IDirectDrawSurface7_Unlock returned 0x%08x, expected DD_OK\n", hr);

1 0

Stefan Dösinger : ddraw: Convert DDSURFACEDESC2 in lock.
by Alexandre Julliard 07 Sep '11

07 Sep '11

Module: wine Branch: master Commit: 029c5a6cf1785901d7f0591da2b54bf639220839 URL: http://source.winehq.org/git/wine.git/?a=commit;h=029c5a6cf1785901d7f0591da… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sun Sep 4 23:33:52 2011 +0200 ddraw: Convert DDSURFACEDESC2 in lock. --- dlls/ddraw/surface.c | 104 +++++++++++++++++++++++++++++++++++--------------- 1 files changed, 73 insertions(+), 31 deletions(-) diff --git a/dlls/ddraw/surface.c b/dlls/ddraw/surface.c index abe3eed..081d4e1 100644 --- a/dlls/ddraw/surface.c +++ b/dlls/ddraw/surface.c @@ -875,18 +875,14 @@ static HRESULT WINAPI ddraw_surface1_GetAttachedSurface(IDirectDrawSurface *ifac * For more details, see IWineD3DSurface::LockRect * *****************************************************************************/ -static HRESULT WINAPI ddraw_surface7_Lock(IDirectDrawSurface7 *iface, +static HRESULT WINAPI surface_lock(IDirectDrawSurfaceImpl *This, RECT *Rect, DDSURFACEDESC2 *DDSD, DWORD Flags, HANDLE h) { - IDirectDrawSurfaceImpl *This = impl_from_IDirectDrawSurface7(iface); WINED3DLOCKED_RECT LockedRect; HRESULT hr; - TRACE("iface %p, rect %s, surface_desc %p, flags %#x, h %p.\n", - iface, wine_dbgstr_rect(Rect), DDSD, Flags, h); - - if(!DDSD) - return DDERR_INVALIDPARAMS; + TRACE("This %p, rect %s, surface_desc %p, flags %#x, h %p.\n", + This, wine_dbgstr_rect(Rect), DDSD, Flags, h); /* This->surface_desc.dwWidth and dwHeight are changeable, thus lock */ EnterCriticalSection(&ddraw_cs); @@ -897,14 +893,6 @@ static HRESULT WINAPI ddraw_surface7_Lock(IDirectDrawSurface7 *iface, * for the supported values. The others are ignored by WineD3D */ - if(DDSD->dwSize != sizeof(DDSURFACEDESC) && - DDSD->dwSize != sizeof(DDSURFACEDESC2)) - { - WARN("Invalid structure size %d, returning DDERR_INVALIDPARAMS\n", DDSD->dwSize); - LeaveCriticalSection(&ddraw_cs); - return DDERR_INVALIDPARAMS; - } - /* Windows zeroes this if the rect is invalid */ DDSD->lpSurface = 0; @@ -954,6 +942,23 @@ static HRESULT WINAPI ddraw_surface7_Lock(IDirectDrawSurface7 *iface, return DD_OK; } +static HRESULT WINAPI ddraw_surface7_Lock(IDirectDrawSurface7 *iface, + RECT *rect, DDSURFACEDESC2 *surface_desc, DWORD flags, HANDLE h) +{ + IDirectDrawSurfaceImpl *This = impl_from_IDirectDrawSurface7(iface); + TRACE("iface %p, rect %s, surface_desc %p, flags %#x, h %p.\n", + iface, wine_dbgstr_rect(rect), surface_desc, flags, h); + + if (!surface_desc) return DDERR_INVALIDPARAMS; + if (surface_desc->dwSize != sizeof(DDSURFACEDESC) && + surface_desc->dwSize != sizeof(DDSURFACEDESC2)) + { + WARN("Invalid structure size %d, returning DDERR_INVALIDPARAMS\n", surface_desc->dwSize); + return DDERR_INVALIDPARAMS; + } + return surface_lock(This, rect, surface_desc, flags, h); +} + static HRESULT WINAPI ddraw_surface4_Lock(IDirectDrawSurface4 *iface, RECT *rect, DDSURFACEDESC2 *surface_desc, DWORD flags, HANDLE h) { @@ -961,49 +966,86 @@ static HRESULT WINAPI ddraw_surface4_Lock(IDirectDrawSurface4 *iface, RECT *rect TRACE("iface %p, rect %s, surface_desc %p, flags %#x, h %p.\n", iface, wine_dbgstr_rect(rect), surface_desc, flags, h); - return ddraw_surface7_Lock(&This->IDirectDrawSurface7_iface, - rect, surface_desc, flags, h); + if (!surface_desc) return DDERR_INVALIDPARAMS; + if (surface_desc->dwSize != sizeof(DDSURFACEDESC) && + surface_desc->dwSize != sizeof(DDSURFACEDESC2)) + { + WARN("Invalid structure size %d, returning DDERR_INVALIDPARAMS\n", surface_desc->dwSize); + return DDERR_INVALIDPARAMS; + } + return surface_lock(This, rect, surface_desc, flags, h); } static HRESULT WINAPI ddraw_surface3_Lock(IDirectDrawSurface3 *iface, RECT *rect, DDSURFACEDESC *surface_desc, DWORD flags, HANDLE h) { IDirectDrawSurfaceImpl *This = impl_from_IDirectDrawSurface3(iface); + DDSURFACEDESC2 surface_desc2; + HRESULT hr; TRACE("iface %p, rect %s, surface_desc %p, flags %#x, h %p.\n", iface, wine_dbgstr_rect(rect), surface_desc, flags, h); - /* All versions of Lock() accept both sizeof(DDSURFACEDESC) and - * sizeof(DDSURFACEDESC2) structures and do not touch the dwSize member */ - return ddraw_surface7_Lock(&This->IDirectDrawSurface7_iface, - rect, (DDSURFACEDESC2 *)surface_desc, flags, h); + if (!surface_desc) return DDERR_INVALIDPARAMS; + if (surface_desc->dwSize != sizeof(DDSURFACEDESC) && + surface_desc->dwSize != sizeof(DDSURFACEDESC2)) + { + WARN("Invalid structure size %d, returning DDERR_INVALIDPARAMS\n", surface_desc->dwSize); + return DDERR_INVALIDPARAMS; + } + + surface_desc2.dwSize = surface_desc->dwSize; + hr = surface_lock(This, rect, &surface_desc2, flags, h); + DDSD2_to_DDSD(&surface_desc2, surface_desc); + surface_desc->dwSize = surface_desc2.dwSize; + return hr; } static HRESULT WINAPI ddraw_surface2_Lock(IDirectDrawSurface2 *iface, RECT *rect, DDSURFACEDESC *surface_desc, DWORD flags, HANDLE h) { IDirectDrawSurfaceImpl *This = impl_from_IDirectDrawSurface2(iface); + DDSURFACEDESC2 surface_desc2; + HRESULT hr; TRACE("iface %p, rect %s, surface_desc %p, flags %#x, h %p.\n", iface, wine_dbgstr_rect(rect), surface_desc, flags, h); - /* All versions of Lock() accept both sizeof(DDSURFACEDESC) and - * sizeof(DDSURFACEDESC2) structures and do not touch the dwSize member - * - * TODO: Test DDSD_ZBUFFERBITDEPTH behavior */ - return ddraw_surface7_Lock(&This->IDirectDrawSurface7_iface, - rect, (DDSURFACEDESC2 *)surface_desc, flags, h); + if (!surface_desc) return DDERR_INVALIDPARAMS; + if (surface_desc->dwSize != sizeof(DDSURFACEDESC) && + surface_desc->dwSize != sizeof(DDSURFACEDESC2)) + { + WARN("Invalid structure size %d, returning DDERR_INVALIDPARAMS\n", surface_desc->dwSize); + return DDERR_INVALIDPARAMS; + } + + surface_desc2.dwSize = surface_desc->dwSize; + hr = surface_lock(This, rect, &surface_desc2, flags, h); + DDSD2_to_DDSD(&surface_desc2, surface_desc); + surface_desc->dwSize = surface_desc2.dwSize; + return hr; } static HRESULT WINAPI ddraw_surface1_Lock(IDirectDrawSurface *iface, RECT *rect, DDSURFACEDESC *surface_desc, DWORD flags, HANDLE h) { IDirectDrawSurfaceImpl *This = impl_from_IDirectDrawSurface(iface); + DDSURFACEDESC2 surface_desc2; + HRESULT hr; TRACE("iface %p, rect %s, surface_desc %p, flags %#x, h %p.\n", iface, wine_dbgstr_rect(rect), surface_desc, flags, h); - /* All versions of Lock() accept both sizeof(DDSURFACEDESC) and - * sizeof(DDSURFACEDESC2) structures and do not touch the dwSize member */ - return ddraw_surface7_Lock(&This->IDirectDrawSurface7_iface, - rect, (DDSURFACEDESC2 *)surface_desc, flags, h); + if (!surface_desc) return DDERR_INVALIDPARAMS; + if (surface_desc->dwSize != sizeof(DDSURFACEDESC) && + surface_desc->dwSize != sizeof(DDSURFACEDESC2)) + { + WARN("Invalid structure size %d, returning DDERR_INVALIDPARAMS\n", surface_desc->dwSize); + return DDERR_INVALIDPARAMS; + } + + surface_desc2.dwSize = surface_desc->dwSize; + hr = surface_lock(This, rect, &surface_desc2, flags, h); + DDSD2_to_DDSD(&surface_desc2, surface_desc); + surface_desc->dwSize = surface_desc2.dwSize; + return hr; } /*****************************************************************************

1 0

Stefan Dösinger : ddraw: Add tests for DDSD_PIXELFORMAT and DDSD_ZBUFFERBITDEPTH in Lock.
by Alexandre Julliard 07 Sep '11

07 Sep '11

Module: wine Branch: master Commit: 4466d6223b3810c3b4341942a1bf2c040558a901 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4466d6223b3810c3b4341942a… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sun Aug 28 19:14:11 2011 +0200 ddraw: Add tests for DDSD_PIXELFORMAT and DDSD_ZBUFFERBITDEPTH in Lock. --- dlls/ddraw/tests/dsurface.c | 39 +++++++++++++++++++++++++++++++++++++++ 1 files changed, 39 insertions(+), 0 deletions(-) diff --git a/dlls/ddraw/tests/dsurface.c b/dlls/ddraw/tests/dsurface.c index a300d5e..a307579 100644 --- a/dlls/ddraw/tests/dsurface.c +++ b/dlls/ddraw/tests/dsurface.c @@ -4184,6 +4184,45 @@ static void test_ddsd(DDSURFACEDESC *ddsd, BOOL expect_pf, BOOL expect_zd, const ok(!(out.dwFlags & DDSD_ZBUFFERBITDEPTH), "%s surface: Expected DDSD_ZBUFFERBITDEPTH not to be set\n", name); } + reset_ddsd(&out); + hr = IDirectDrawSurface_Lock(surface, NULL, &out, 0, NULL); + if (SUCCEEDED(hr)) + { + hr = IDirectDrawSurface_Unlock(surface, NULL); + ok(SUCCEEDED(hr), "IDirectDrawSurface_GetSurfaceDesc failed, hr %#x.\n", hr); + + /* DDSD_ZBUFFERBITDEPTH is never set on Nvidia, but follows GetSurfaceDesc rules on AMD */ + if (!expect_zd) + { + ok(!(out.dwFlags & DDSD_ZBUFFERBITDEPTH), + "Lock %s surface: Expected DDSD_ZBUFFERBITDEPTH not to be set\n", name); + } + + /* DDSD_PIXELFORMAT follows GetSurfaceDesc rules */ + if (expect_pf) + { + ok(out.dwFlags & DDSD_PIXELFORMAT, "%s surface: Expected DDSD_PIXELFORMAT to be set\n", name); + } + else + { + ok(!(out.dwFlags & DDSD_PIXELFORMAT), + "Lock %s surface: Expected DDSD_PIXELFORMAT not to be set\n", name); + } + } + + hr = IDirectDrawSurface7_Lock(surface7, NULL, &out2, 0, NULL); + ok(SUCCEEDED(hr), "IDirectDrawSurface7_Lock failed, hr %#x.\n", hr); + if (SUCCEEDED(hr)) + { + hr = IDirectDrawSurface7_Unlock(surface7, NULL); + ok(SUCCEEDED(hr), "IDirectDrawSurface7_Unlock failed, hr %#x.\n", hr); + /* DDSD_PIXELFORMAT is always set, DDSD_ZBUFFERBITDEPTH never */ + ok(out2.dwFlags & DDSD_PIXELFORMAT, + "Lock %s surface: Expected DDSD_PIXELFORMAT to be set in DDSURFACEDESC2\n", name); + ok(!(out2.dwFlags & DDSD_ZBUFFERBITDEPTH), + "Lock %s surface: Did not expect DDSD_ZBUFFERBITDEPTH to be set in DDSURFACEDESC2\n", name); + } + IDirectDrawSurface7_Release(surface7); IDirectDrawSurface_Release(surface); }

1 0

Stefan Dösinger : ddraw: Add tests for dwZBufferBitDepth and dwMipMapCount values.
by Alexandre Julliard 07 Sep '11

07 Sep '11

Module: wine Branch: master Commit: 73f9dd1c30a0046ebd7be9934f8dc952bf7d4c3b URL: http://source.winehq.org/git/wine.git/?a=commit;h=73f9dd1c30a0046ebd7be9934… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sun Sep 4 23:41:41 2011 +0200 ddraw: Add tests for dwZBufferBitDepth and dwMipMapCount values. --- dlls/ddraw/tests/dsurface.c | 24 ++++++++++++++++++++---- 1 files changed, 20 insertions(+), 4 deletions(-) diff --git a/dlls/ddraw/tests/dsurface.c b/dlls/ddraw/tests/dsurface.c index a307579..6b23c70 100644 --- a/dlls/ddraw/tests/dsurface.c +++ b/dlls/ddraw/tests/dsurface.c @@ -4137,7 +4137,7 @@ static void zbufferbitdepth_test(void) U2(ddsd).dwZBufferBitDepth); } -static void test_ddsd(DDSURFACEDESC *ddsd, BOOL expect_pf, BOOL expect_zd, const char *name) +static void test_ddsd(DDSURFACEDESC *ddsd, BOOL expect_pf, BOOL expect_zd, const char *name, DWORD z_bit_depth) { IDirectDrawSurface *surface; IDirectDrawSurface7 *surface7; @@ -4166,6 +4166,8 @@ static void test_ddsd(DDSURFACEDESC *ddsd, BOOL expect_pf, BOOL expect_zd, const if (expect_pf) { ok(out.dwFlags & DDSD_PIXELFORMAT, "%s surface: Expected DDSD_PIXELFORMAT to be set\n", name); + ok(out2.dwFlags & DDSD_PIXELFORMAT, + "%s surface: Expected DDSD_PIXELFORMAT to be set in DDSURFACEDESC2\n", name); } else { @@ -4176,12 +4178,20 @@ static void test_ddsd(DDSURFACEDESC *ddsd, BOOL expect_pf, BOOL expect_zd, const if (expect_zd) { ok(out.dwFlags & DDSD_ZBUFFERBITDEPTH, "%s surface: Expected DDSD_ZBUFFERBITDEPTH to be set\n", name); + ok(U2(out).dwZBufferBitDepth == z_bit_depth, "ZBufferBitDepth is %u, expected %u\n", + U2(out).dwZBufferBitDepth, z_bit_depth); ok(!(out2.dwFlags & DDSD_ZBUFFERBITDEPTH), "%s surface: Did not expect DDSD_ZBUFFERBITDEPTH to be set in DDSURFACEDESC2\n", name); + /* dwMipMapCount and dwZBufferBitDepth share the same union */ + ok(U2(out2).dwMipMapCount == 0, "dwMipMapCount is %u, expected 0\n", U2(out2).dwMipMapCount); } else { ok(!(out.dwFlags & DDSD_ZBUFFERBITDEPTH), "%s surface: Expected DDSD_ZBUFFERBITDEPTH not to be set\n", name); + ok(U2(out).dwZBufferBitDepth == 0, "ZBufferBitDepth is %u, expected 0\n", U2(out).dwZBufferBitDepth); + ok(!(out2.dwFlags & DDSD_ZBUFFERBITDEPTH), + "%s surface: Did not expect DDSD_ZBUFFERBITDEPTH to be set in DDSURFACEDESC2\n", name); + ok(U2(out2).dwMipMapCount == 0, "dwMipMapCount is %u, expected 0\n", U2(out2).dwMipMapCount); } reset_ddsd(&out); @@ -4208,6 +4218,11 @@ static void test_ddsd(DDSURFACEDESC *ddsd, BOOL expect_pf, BOOL expect_zd, const ok(!(out.dwFlags & DDSD_PIXELFORMAT), "Lock %s surface: Expected DDSD_PIXELFORMAT not to be set\n", name); } + if (out.dwFlags & DDSD_ZBUFFERBITDEPTH) + ok(U2(out).dwZBufferBitDepth == z_bit_depth, "ZBufferBitDepth is %u, expected %u\n", + U2(out).dwZBufferBitDepth, z_bit_depth); + else + ok(U2(out).dwZBufferBitDepth == 0, "ZBufferBitDepth is %u, expected 0\n", U2(out).dwZBufferBitDepth); } hr = IDirectDrawSurface7_Lock(surface7, NULL, &out2, 0, NULL); @@ -4221,6 +4236,7 @@ static void test_ddsd(DDSURFACEDESC *ddsd, BOOL expect_pf, BOOL expect_zd, const "Lock %s surface: Expected DDSD_PIXELFORMAT to be set in DDSURFACEDESC2\n", name); ok(!(out2.dwFlags & DDSD_ZBUFFERBITDEPTH), "Lock %s surface: Did not expect DDSD_ZBUFFERBITDEPTH to be set in DDSURFACEDESC2\n", name); + ok(U2(out2).dwMipMapCount == 0, "dwMipMapCount is %u, expected 0\n", U2(out2).dwMipMapCount); } IDirectDrawSurface7_Release(surface7); @@ -4248,12 +4264,12 @@ static void pixelformat_flag_test(void) ddsd.dwWidth = 64; ddsd.dwHeight = 64; ddsd.ddsCaps.dwCaps = DDSCAPS_OFFSCREENPLAIN; - test_ddsd(&ddsd, TRUE, FALSE, "offscreen plain"); + test_ddsd(&ddsd, TRUE, FALSE, "offscreen plain", ~0U); reset_ddsd(&ddsd); ddsd.dwFlags = DDSD_CAPS; ddsd.ddsCaps.dwCaps = DDSCAPS_PRIMARYSURFACE; - test_ddsd(&ddsd, TRUE, FALSE, "primary"); + test_ddsd(&ddsd, TRUE, FALSE, "primary", ~0U); reset_ddsd(&ddsd); ddsd.dwFlags = DDSD_CAPS | DDSD_WIDTH | DDSD_HEIGHT | DDSD_ZBUFFERBITDEPTH; @@ -4261,7 +4277,7 @@ static void pixelformat_flag_test(void) ddsd.dwHeight = 64; U2(ddsd).dwZBufferBitDepth = 16; ddsd.ddsCaps.dwCaps = DDSCAPS_ZBUFFER; - test_ddsd(&ddsd, FALSE, TRUE, "Z buffer"); + test_ddsd(&ddsd, FALSE, TRUE, "Z buffer", 16); } static void set_surface_desc_test(void)

1 0

Alistair Leslie-Hughes : msxml3: Correct Objsafe tests for older msxml3.
by Alexandre Julliard 07 Sep '11

07 Sep '11

Module: wine Branch: master Commit: e108c2dbed783982132aa19fa3cfb8bb7fce230f URL: http://source.winehq.org/git/wine.git/?a=commit;h=e108c2dbed783982132aa19fa… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Wed Sep 7 08:25:54 2011 +1000 msxml3: Correct Objsafe tests for older msxml3. --- dlls/msxml3/tests/domdoc.c | 15 ++++++++------- 1 files changed, 8 insertions(+), 7 deletions(-) diff --git a/dlls/msxml3/tests/domdoc.c b/dlls/msxml3/tests/domdoc.c index 42b3b05..22fb3ae 100644 --- a/dlls/msxml3/tests/domdoc.c +++ b/dlls/msxml3/tests/domdoc.c @@ -4458,10 +4458,10 @@ static void _test_IObjectSafety_common(unsigned line, IObjectSafety *safety) hr = IObjectSafety_GetInterfaceSafetyOptions(safety, NULL, &supported, &enabled); ok_(__FILE__,line)(hr == S_OK, "ret %08x\n", hr ); ok_(__FILE__,line)(enabled == INTERFACESAFE_FOR_UNTRUSTED_CALLER, "Expected INTERFACESAFE_FOR_UNTRUSTED_CALLER got %08x\n", enabled); - ok_(__FILE__,line)(broken(supported == INTERFACESAFE_FOR_UNTRUSTED_CALLER) || + ok_(__FILE__,line)(broken(supported == (INTERFACESAFE_FOR_UNTRUSTED_CALLER | INTERFACESAFE_FOR_UNTRUSTED_DATA)) || supported == (INTERFACESAFE_FOR_UNTRUSTED_CALLER | INTERFACESAFE_FOR_UNTRUSTED_DATA | INTERFACE_USES_SECURITY_MANAGER) /* msxml3 SP8+ */, - "Expected (INTERFACESAFE_FOR_UNTRUSTED_CALLER | INTERFACESAFE_FOR_UNTRUSTED_DATA | INTERFACE_USES_SECURITY_MANAGER), " - "got %08x\n", supported); + "Expected (INTERFACESAFE_FOR_UNTRUSTED_CALLER | INTERFACESAFE_FOR_UNTRUSTED_DATA | INTERFACE_USES_SECURITY_MANAGER), " + "got %08x\n", supported); hr = IObjectSafety_SetInterfaceSafetyOptions(safety, NULL, INTERFACESAFE_FOR_UNTRUSTED_DATA, @@ -4469,12 +4469,13 @@ static void _test_IObjectSafety_common(unsigned line, IObjectSafety *safety) ok_(__FILE__,line)(hr == S_OK, "ret %08x\n", hr ); hr = IObjectSafety_GetInterfaceSafetyOptions(safety, NULL, &supported, &enabled); ok_(__FILE__,line)(hr == S_OK, "ret %08x\n", hr ); - ok_(__FILE__,line)(enabled == (INTERFACESAFE_FOR_UNTRUSTED_CALLER | INTERFACESAFE_FOR_UNTRUSTED_DATA), - "expected INTERFACESAFE_FOR_UNTRUSTED_CALLER | INTERFACESAFE_FOR_UNTRUSTED_DATA got %08x\n", enabled); - ok_(__FILE__,line)(broken(supported == INTERFACESAFE_FOR_UNTRUSTED_DATA) || + ok_(__FILE__,line)(broken(enabled == INTERFACESAFE_FOR_UNTRUSTED_DATA) || + enabled == (INTERFACESAFE_FOR_UNTRUSTED_CALLER | INTERFACESAFE_FOR_UNTRUSTED_DATA), + "Expected (INTERFACESAFE_FOR_UNTRUSTED_CALLER | INTERFACESAFE_FOR_UNTRUSTED_DATA) got %08x\n", enabled); + ok_(__FILE__,line)(broken(supported == (INTERFACESAFE_FOR_UNTRUSTED_CALLER | INTERFACESAFE_FOR_UNTRUSTED_DATA)) || supported == (INTERFACESAFE_FOR_UNTRUSTED_CALLER | INTERFACESAFE_FOR_UNTRUSTED_DATA | INTERFACE_USES_SECURITY_MANAGER) /* msxml3 SP8+ */, "Expected (INTERFACESAFE_FOR_UNTRUSTED_CALLER | INTERFACESAFE_FOR_UNTRUSTED_DATA | INTERFACE_USES_SECURITY_MANAGER), " - "got %08x\n", supported); + "got %08x\n", supported); } static void test_XMLHTTP(void)

1 0

Alexandre Julliard : gdi32: Add RLE support in SetDIBitsToDevice.
by Alexandre Julliard 06 Sep '11

06 Sep '11

Module: wine Branch: master Commit: 6ae21b934595dd286aa3a0a088650ad88e7a671e URL: http://source.winehq.org/git/wine.git/?a=commit;h=6ae21b934595dd286aa3a0a08… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Sep 6 14:36:27 2011 +0200 gdi32: Add RLE support in SetDIBitsToDevice. --- dlls/gdi32/dib.c | 66 +++++++++---- dlls/gdi32/tests/bitmap.c | 222 +++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 268 insertions(+), 20 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=6ae21b934595dd286aa3a…

1 0

Alexandre Julliard : winex11: Get rid of the no longer used RLE support.
by Alexandre Julliard 06 Sep '11

06 Sep '11

Module: wine Branch: master Commit: cd7ccebb9b0c2f3f7bd6f718027bbb5070731914 URL: http://source.winehq.org/git/wine.git/?a=commit;h=cd7ccebb9b0c2f3f7bd6f7180… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Sep 6 14:48:37 2011 +0200 winex11: Get rid of the no longer used RLE support. --- dlls/winex11.drv/dib.c | 261 ++---------------------------------------------- 1 files changed, 8 insertions(+), 253 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=cd7ccebb9b0c2f3f7bd6f…

1 0