March 2020 - Wine-commits - list.winehq.org

Paul Gofman : ddraw/tests: Test locking lost surface for ddraw1.
by Alexandre Julliard 26 Mar '20

26 Mar '20

Module: wine Branch: master Commit: 792dc34978c6bcc4bd08a39cb6b4ef9510e3dabe URL: https://source.winehq.org/git/wine.git/?a=commit;h=792dc34978c6bcc4bd08a39c… Author: Paul Gofman <gofmanp(a)gmail.com> Date: Thu Mar 26 16:09:44 2020 +0300 ddraw/tests: Test locking lost surface for ddraw1. Signed-off-by: Paul Gofman <gofmanp(a)gmail.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ddraw/tests/ddraw1.c | 28 +++++++++++++++++++++------- 1 file changed, 21 insertions(+), 7 deletions(-) diff --git a/dlls/ddraw/tests/ddraw1.c b/dlls/ddraw/tests/ddraw1.c index 5bfca43008..ff100c64aa 100644 --- a/dlls/ddraw/tests/ddraw1.c +++ b/dlls/ddraw/tests/ddraw1.c @@ -1085,6 +1085,7 @@ static void test_coop_level_d3d_state(void) IDirect3DViewport *viewport; IDirect3DDevice *device; D3DMATERIAL material; + DDSURFACEDESC lock; IDirectDraw *ddraw; D3DCOLOR color; HWND window; @@ -1106,18 +1107,31 @@ static void test_coop_level_d3d_state(void) viewport_set_background(device, viewport, background); hr = IDirect3DDevice_QueryInterface(device, &IID_IDirectDrawSurface, (void **)&rt); - ok(SUCCEEDED(hr), "Failed to get render target, hr %#x.\n", hr); + ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr); hr = IDirect3DViewport_Clear(viewport, 1, &clear_rect, D3DCLEAR_TARGET); - ok(SUCCEEDED(hr), "Failed to clear viewport, hr %#x.\n", hr); + ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr); color = get_surface_color(rt, 320, 240); ok(compare_color(color, 0x00ff0000, 1), "Got unexpected color 0x%08x.\n", color); hr = IDirectDraw_SetCooperativeLevel(ddraw, window, DDSCL_EXCLUSIVE | DDSCL_FULLSCREEN); - ok(SUCCEEDED(hr), "Failed to set cooperative level, hr %#x.\n", hr); + ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr); hr = IDirectDrawSurface_IsLost(rt); ok(hr == DDERR_SURFACELOST, "Got unexpected hr %#x.\n", hr); + + memset(&lock, 0, sizeof(lock)); + lock.dwSize = sizeof(lock); + lock.lpSurface = (void *)0xdeadbeef; + hr = IDirectDrawSurface_Lock(rt, NULL, &lock, DDLOCK_READONLY, NULL); + ok(hr == DDERR_SURFACELOST, "Got unexpected hr %#x.\n", hr); + ok(lock.lpSurface == (void *)0xdeadbeef, "Got unexpected lock.lpSurface %p.\n", lock.lpSurface); + hr = restore_surfaces(ddraw); - ok(SUCCEEDED(hr), "Failed to restore surfaces, hr %#x.\n", hr); + ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr); + + hr = IDirectDrawSurface_Lock(rt, NULL, &lock, DDLOCK_READONLY, NULL); + ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr); + hr = IDirectDrawSurface2_Unlock(rt, NULL); + ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr); memset(&material, 0, sizeof(material)); material.dwSize = sizeof(material); @@ -1126,13 +1140,13 @@ static void test_coop_level_d3d_state(void) U3(U(material).diffuse).b = 0.0f; U4(U(material).diffuse).a = 1.0f; hr = IDirect3DMaterial_SetMaterial(background, &material); - ok(SUCCEEDED(hr), "Failed to set material data, hr %#x.\n", hr); + ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr); hr = IDirect3DDevice_QueryInterface(device, &IID_IDirectDrawSurface, (void **)&surface); - ok(SUCCEEDED(hr), "Failed to get render target, hr %#x.\n", hr); + ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr); ok(surface == rt, "Got unexpected surface %p.\n", surface); hr = IDirect3DViewport_Clear(viewport, 1, &clear_rect, D3DCLEAR_TARGET); - ok(SUCCEEDED(hr), "Failed to clear viewport, hr %#x.\n", hr); + ok(hr == DD_OK, "Got unexpected hr %#x.\n", hr); color = get_surface_color(rt, 320, 240); ok(compare_color(color, 0x0000ff00, 1) || broken(compare_color(color, 0x00000000, 1)), "Got unexpected color 0x%08x.\n", color);

1 0

Piotr Caban : ucrtbase/tests: Set invalid_parameter_handler once.
by Alexandre Julliard 26 Mar '20

26 Mar '20

Module: wine Branch: master Commit: e115d5c71844dd4255785279ccc8cbe9b6a07a3b URL: https://source.winehq.org/git/wine.git/?a=commit;h=e115d5c71844dd4255785279… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Thu Mar 26 11:42:00 2020 +0100 ucrtbase/tests: Set invalid_parameter_handler once. Signed-off-by: Piotr Caban <piotr(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ucrtbase/tests/printf.c | 9 +++------ dlls/ucrtbase/tests/string.c | 27 +++------------------------ 2 files changed, 6 insertions(+), 30 deletions(-) diff --git a/dlls/ucrtbase/tests/printf.c b/dlls/ucrtbase/tests/printf.c index d6a1aa048e..8cacbbb79a 100644 --- a/dlls/ucrtbase/tests/printf.c +++ b/dlls/ucrtbase/tests/printf.c @@ -376,9 +376,6 @@ static void test_fwprintf(void) fclose(fp); unlink(file_name); - ok(_set_invalid_parameter_handler(test_invalid_parameter_handler) == NULL, - "Invalid parameter handler was already set\n"); - /* NULL format */ errno = 0xdeadbeef; SET_EXPECT(invalid_parameter_handler); @@ -398,9 +395,6 @@ static void test_fwprintf(void) /* format using % with NULL arglist*/ /* crashes on Windows */ /* ret = __stdio_common_vfwprintf(0, fp, cont_fmt, NULL, NULL); */ - - ok(_set_invalid_parameter_handler(NULL) == test_invalid_parameter_handler, - "Cannot reset invalid parameter handler\n"); } static int WINAPIV _vsnprintf_s_wrapper(char *str, size_t sizeOfBuffer, @@ -628,6 +622,9 @@ static void test_printf_natural_string(void) START_TEST(printf) { + ok(_set_invalid_parameter_handler(test_invalid_parameter_handler) == NULL, + "Invalid parameter handler was already set\n"); + test_snprintf(); test_swprintf(); test_fprintf(); diff --git a/dlls/ucrtbase/tests/string.c b/dlls/ucrtbase/tests/string.c index 74f0f56be0..6e6f1e80e2 100644 --- a/dlls/ucrtbase/tests/string.c +++ b/dlls/ucrtbase/tests/string.c @@ -150,9 +150,6 @@ static void test__memicmp(void) static const char *s2 = "aBd"; int ret; - ok(_set_invalid_parameter_handler(test_invalid_parameter_handler) == NULL, - "Invalid parameter handler was already set\n"); - ret = _memicmp(NULL, NULL, 0); ok(!ret, "got %d\n", ret); @@ -182,9 +179,6 @@ static void test__memicmp(void) ret = _memicmp(s1, s2, 3); ok(ret == -1, "got %d\n", ret); - - ok(_set_invalid_parameter_handler(NULL) == test_invalid_parameter_handler, - "Cannot reset invalid parameter handler\n"); } static void test__memicmp_l(void) @@ -193,9 +187,6 @@ static void test__memicmp_l(void) static const char *s2 = "aBd"; int ret; - ok(_set_invalid_parameter_handler(test_invalid_parameter_handler) == NULL, - "Invalid parameter handler was already set\n"); - ret = _memicmp_l(NULL, NULL, 0, NULL); ok(!ret, "got %d\n", ret); @@ -225,9 +216,6 @@ static void test__memicmp_l(void) ret = _memicmp_l(s1, s2, 3, NULL); ok(ret == -1, "got %d\n", ret); - - ok(_set_invalid_parameter_handler(NULL) == test_invalid_parameter_handler, - "Cannot reset invalid parameter handler\n"); } @@ -255,17 +243,11 @@ static void test___strncnt(void) ok(ret == strncnt_tests[i].ret, "%u: unexpected return value %u.\n", i, (int)ret); } - ok(_set_invalid_parameter_handler(test_invalid_parameter_handler) == NULL, - "Invalid parameter handler was already set\n"); - if (0) /* crashes */ { ret = __strncnt(NULL, 0); ret = __strncnt(NULL, 1); } - - ok(_set_invalid_parameter_handler(NULL) == test_invalid_parameter_handler, - "Cannot reset invalid parameter handler\n"); } static void test_C_locale(void) @@ -436,9 +418,6 @@ static void test__strnicmp(void) static const char str2[] = "test"; int ret; - ok(_set_invalid_parameter_handler(test_invalid_parameter_handler) == NULL, - "Invalid parameter handler was already set\n"); - SET_EXPECT(invalid_parameter_handler); errno = 0xdeadbeef; ret = _strnicmp(str1, str2, -1); @@ -448,13 +427,13 @@ static void test__strnicmp(void) ret = _strnicmp(str1, str2, 0x7fffffff); ok(!ret, "got %d.\n", ret); - - ok(_set_invalid_parameter_handler(NULL) == test_invalid_parameter_handler, - "Cannot reset invalid parameter handler\n"); } START_TEST(string) { + ok(_set_invalid_parameter_handler(test_invalid_parameter_handler) == NULL, + "Invalid parameter handler was already set\n"); + test_strtod(); test__memicmp(); test__memicmp_l();

1 0

Nikolay Sivov : mf: Clean traces format for topology methods.
by Alexandre Julliard 26 Mar '20

26 Mar '20

Module: wine Branch: master Commit: 979c9b2ba2a5a04ff39b2d07a4cd541a43492509 URL: https://source.winehq.org/git/wine.git/?a=commit;h=979c9b2ba2a5a04ff39b2d07… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Mar 26 16:17:15 2020 +0300 mf: Clean traces format for topology methods. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mf/topology.c | 166 ++++++++++++++++++++++++++--------------------------- 1 file changed, 83 insertions(+), 83 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=979c9b2ba2a5a04ff39b…

1 0

Nikolay Sivov : mf: Fix a typo when setting event attribute.
by Alexandre Julliard 26 Mar '20

26 Mar '20

Module: wine Branch: master Commit: 875b0c40a9e9ce208223a9d3c43bf805a20c7fab URL: https://source.winehq.org/git/wine.git/?a=commit;h=875b0c40a9e9ce208223a9d3… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Mar 26 16:53:31 2020 +0300 mf: Fix a typo when setting event attribute. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mf/session.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/mf/session.c b/dlls/mf/session.c index 50f011dc52..2147f70cc8 100644 --- a/dlls/mf/session.c +++ b/dlls/mf/session.c @@ -1189,7 +1189,7 @@ static HRESULT session_set_current_topology(struct media_session *session, IMFTo { IMFMediaEvent_SetUINT64(event, &MF_EVENT_START_PRESENTATION_TIME, 0); IMFMediaEvent_SetUINT64(event, &MF_EVENT_PRESENTATION_TIME_OFFSET, 0); - IMFMediaEvent_GetUINT64(event, &MF_EVENT_START_PRESENTATION_TIME_AT_OUTPUT, 0); + IMFMediaEvent_SetUINT64(event, &MF_EVENT_START_PRESENTATION_TIME_AT_OUTPUT, 0); IMFMediaEventQueue_QueueEvent(session->event_queue, event); IMFMediaEvent_Release(event);

1 0

Jacek Caban : dbghelp: Remove unused code in module_find_cb.
by Alexandre Julliard 26 Mar '20

26 Mar '20

Module: wine Branch: master Commit: 8d702a3aa54fc61bd146f584e66364a12a0ee3a0 URL: https://source.winehq.org/git/wine.git/?a=commit;h=8d702a3aa54fc61bd146f584… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Mar 26 16:51:20 2020 +0100 dbghelp: Remove unused code in module_find_cb. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dbghelp/path.c | 26 +------------------------- 1 file changed, 1 insertion(+), 25 deletions(-) diff --git a/dlls/dbghelp/path.c b/dlls/dbghelp/path.c index 06208d109d..24649cf280 100644 --- a/dlls/dbghelp/path.c +++ b/dlls/dbghelp/path.c @@ -474,7 +474,7 @@ struct module_find static BOOL CALLBACK module_find_cb(PCWSTR buffer, PVOID user) { struct module_find* mf = user; - DWORD size, checksum, timestamp; + DWORD size, timestamp; unsigned matched = 0; /* the matching weights: @@ -524,30 +524,6 @@ static BOOL CALLBACK module_find_cb(PCWSTR buffer, PVOID user) if (timestamp == mf->dw1 && size == mf->dw2) matched++; } break; - case DMT_MACHO: - case DMT_ELF: - { - HANDLE file; - - file = CreateFileW(buffer, GENERIC_READ, FILE_SHARE_READ, NULL, - OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL); - if (file == INVALID_HANDLE_VALUE) break; - - checksum = calc_crc32(file); - if (checksum == mf->dw1) matched += 2; - else - { - struct image_file_map fmap; - WARN("Found %s, but wrong checksums: %08x %08x\n", debugstr_w(buffer), checksum, mf->dw1); - if (elf_map_handle(file, &fmap)) /* FIXME: validate macho files */ - { - image_unmap_file(&fmap); - matched++; - } - } - CloseHandle(file); - } - break; case DMT_PDB: { struct pdb_lookup pdb_lookup;

1 0

Jacek Caban : dbghelp: Use LONG_PTR instead of long type.
by Alexandre Julliard 26 Mar '20

26 Mar '20

Module: wine Branch: master Commit: f803da493aeb4cf118fb9b076e8042c2535fbcdb URL: https://source.winehq.org/git/wine.git/?a=commit;h=f803da493aeb4cf118fb9b07… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Mar 26 16:51:49 2020 +0100 dbghelp: Use LONG_PTR instead of long type. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dbghelp/cpu_i386.c | 2 +- dlls/dbghelp/dbghelp_private.h | 68 ++++++++++++++-------------- dlls/dbghelp/dwarf.c | 100 ++++++++++++++++++++--------------------- dlls/dbghelp/elf_module.c | 50 ++++++++++----------- dlls/dbghelp/image_private.h | 2 +- dlls/dbghelp/macho_module.c | 70 ++++++++++++++--------------- dlls/dbghelp/minidump.c | 2 +- dlls/dbghelp/module.c | 4 +- dlls/dbghelp/msc.c | 6 +-- dlls/dbghelp/stabs.c | 54 +++++++++++----------- dlls/dbghelp/storage.c | 8 ++-- dlls/dbghelp/symbol.c | 18 ++++---- dlls/dbghelp/type.c | 2 +- 13 files changed, 193 insertions(+), 193 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=f803da493aeb4cf118fb…

1 0

Matteo Bruni : wined3d: Fix ARB_TEXTURE_STORAGE_MULTISAMPLE core_extension[] entry.
by Alexandre Julliard 26 Mar '20

26 Mar '20

Module: wine Branch: master Commit: a281544bde79592bdd128e8691ce310ce260c3da URL: https://source.winehq.org/git/wine.git/?a=commit;h=a281544bde79592bdd128e86… Author: Matteo Bruni <mbruni(a)codeweavers.com> Date: Thu Mar 26 13:25:59 2020 +0100 wined3d: Fix ARB_TEXTURE_STORAGE_MULTISAMPLE core_extension[] entry. Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/adapter_gl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/wined3d/adapter_gl.c b/dlls/wined3d/adapter_gl.c index 33ba2c01ca..77fe4ca720 100644 --- a/dlls/wined3d/adapter_gl.c +++ b/dlls/wined3d/adapter_gl.c @@ -3423,7 +3423,7 @@ static BOOL wined3d_adapter_init_gl_caps(struct wined3d_adapter *adapter, {ARB_STENCIL_TEXTURING, MAKEDWORD_VERSION(4, 3)}, {ARB_TEXTURE_BUFFER_RANGE, MAKEDWORD_VERSION(4, 3)}, {ARB_TEXTURE_QUERY_LEVELS, MAKEDWORD_VERSION(4, 3)}, - {ARB_TEXTURE_STORAGE_MULTISAMPLE, MAKEDWORD_VERSION(4, 2)}, + {ARB_TEXTURE_STORAGE_MULTISAMPLE, MAKEDWORD_VERSION(4, 3)}, {ARB_TEXTURE_VIEW, MAKEDWORD_VERSION(4, 3)}, {ARB_BUFFER_STORAGE, MAKEDWORD_VERSION(4, 4)},

1 0

Matteo Bruni : wined3d: Skip loading current index buffer if the draw is not indexed.
by Alexandre Julliard 26 Mar '20

26 Mar '20

Module: wine Branch: master Commit: b83b897b4a2f96be7446a7b8ac6678b1559e9386 URL: https://source.winehq.org/git/wine.git/?a=commit;h=b83b897b4a2f96be7446a7b8… Author: Matteo Bruni <mbruni(a)codeweavers.com> Date: Thu Mar 26 13:26:00 2020 +0100 wined3d: Skip loading current index buffer if the draw is not indexed. Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/context.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index c6a1d76267..2e363fb1d6 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -4225,7 +4225,7 @@ static void context_load_stream_output_buffers(struct wined3d_context *context, /* Context activation is done by the caller. */ static BOOL context_apply_draw_state(struct wined3d_context *context, - const struct wined3d_device *device, const struct wined3d_state *state) + const struct wined3d_device *device, const struct wined3d_state *state, BOOL indexed) { const struct wined3d_state_entry *state_table = context->state_table; struct wined3d_context_gl *context_gl = wined3d_context_gl(context); @@ -4277,7 +4277,7 @@ static BOOL context_apply_draw_state(struct wined3d_context *context, if (isStateDirty(context, STATE_STREAMSRC)) context_update_stream_info(context, state); } - if (state->index_buffer) + if (indexed && state->index_buffer) { if (context->stream_info.all_vbo) wined3d_buffer_load(state->index_buffer, context, state); @@ -5194,7 +5194,7 @@ void draw_primitive(struct wined3d_device *device, const struct wined3d_state *s if (parameters->indirect) wined3d_buffer_load(parameters->u.indirect.buffer, context, state); - if (!context_apply_draw_state(context, device, state)) + if (!context_apply_draw_state(context, device, state, parameters->indexed)) { context_release(context); WARN("Unable to apply draw state, skipping draw.\n");

1 0

Matteo Bruni : wined3d: Only invalidate STATE_INDEXBUFFER if the buffer being destroyed is bound as index buffer.
by Alexandre Julliard 26 Mar '20

26 Mar '20

Module: wine Branch: master Commit: 3cc218d5b51140354afdd689c67be9258476a1e3 URL: https://source.winehq.org/git/wine.git/?a=commit;h=3cc218d5b51140354afdd689… Author: Matteo Bruni <mbruni(a)codeweavers.com> Date: Thu Mar 26 13:26:01 2020 +0100 wined3d: Only invalidate STATE_INDEXBUFFER if the buffer being destroyed is bound as index buffer. Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/buffer.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/dlls/wined3d/buffer.c b/dlls/wined3d/buffer.c index 427b1a2649..dffa703dcc 100644 --- a/dlls/wined3d/buffer.c +++ b/dlls/wined3d/buffer.c @@ -144,6 +144,8 @@ static void wined3d_buffer_gl_destroy_buffer_object(struct wined3d_buffer_gl *bu { const struct wined3d_gl_info *gl_info = context_gl->gl_info; struct wined3d_resource *resource = &buffer_gl->b.resource; + struct wined3d_buffer *buffer = &buffer_gl->b; + struct wined3d_cs *cs = resource->device->cs; GLuint bo; if (!buffer_gl->b.buffer_object) @@ -158,7 +160,8 @@ static void wined3d_buffer_gl_destroy_buffer_object(struct wined3d_buffer_gl *bu { if (resource->bind_flags & WINED3D_BIND_VERTEX_BUFFER) device_invalidate_state(resource->device, STATE_STREAMSRC); - if (resource->bind_flags & WINED3D_BIND_INDEX_BUFFER) + if (resource->bind_flags & WINED3D_BIND_INDEX_BUFFER + && cs->state.index_buffer == buffer) device_invalidate_state(resource->device, STATE_INDEXBUFFER); if (resource->bind_flags & WINED3D_BIND_CONSTANT_BUFFER) {

1 0

Paul Gofman : ddraw: Factor out ddraw_surface_is_lost() function.
by Alexandre Julliard 26 Mar '20

26 Mar '20

Module: wine Branch: master Commit: d36a45e55387efc017b32da3ba7fdd9e8f505aa6 URL: https://source.winehq.org/git/wine.git/?a=commit;h=d36a45e55387efc017b32da3… Author: Paul Gofman <gofmanp(a)gmail.com> Date: Thu Mar 26 16:09:39 2020 +0300 ddraw: Factor out ddraw_surface_is_lost() function. Signed-off-by: Paul Gofman <gofmanp(a)gmail.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ddraw/surface.c | 34 ++++++++++++++++++---------------- 1 file changed, 18 insertions(+), 16 deletions(-) diff --git a/dlls/ddraw/surface.c b/dlls/ddraw/surface.c index 85c1ef4966..9d937414ae 100644 --- a/dlls/ddraw/surface.c +++ b/dlls/ddraw/surface.c @@ -33,6 +33,11 @@ static inline struct ddraw_surface *impl_from_IDirectDrawGammaControl(IDirectDra return CONTAINING_RECORD(iface, struct ddraw_surface, IDirectDrawGammaControl_iface); } +static BOOL ddraw_surface_is_lost(const struct ddraw_surface *surface) +{ + return surface->ddraw->device_state != DDRAW_DEVICE_STATE_OK || surface->is_lost; +} + /* This is slow, of course. Also, in case of locks, we can't prevent other * applications from drawing to the screen while we've locked the frontbuffer. * We'd like to do this in wined3d instead, but for that to work wined3d needs @@ -766,7 +771,7 @@ static HRESULT WINAPI ddraw_surface7_GetAttachedSurface(IDirectDrawSurface7 *ifa TRACE("iface %p, caps %p, attachment %p.\n", iface, caps, surface); - if (IDirectDrawSurface7_IsLost(&head_surface->IDirectDrawSurface7_iface) != DD_OK) + if (ddraw_surface_is_lost(head_surface)) { WARN("Surface %p is lost.\n", head_surface); @@ -1291,7 +1296,7 @@ static HRESULT WINAPI DECLSPEC_HOTPATCH ddraw_surface7_Flip(IDirectDrawSurface7 if (src == iface || !(dst_impl->surface_desc.ddsCaps.dwCaps & (DDSCAPS_FRONTBUFFER | DDSCAPS_OVERLAY))) return DDERR_NOTFLIPPABLE; - if (IDirectDrawSurface7_IsLost(iface) == DDERR_SURFACELOST) + if (ddraw_surface_is_lost(dst_impl)) return DDERR_SURFACELOST; wined3d_mutex_lock(); @@ -3681,10 +3686,7 @@ static HRESULT WINAPI ddraw_surface7_IsLost(IDirectDrawSurface7 *iface) TRACE("iface %p.\n", iface); - if (surface->ddraw->device_state != DDRAW_DEVICE_STATE_OK || surface->is_lost) - return DDERR_SURFACELOST; - - return DD_OK; + return ddraw_surface_is_lost(surface) ? DDERR_SURFACELOST : DD_OK; } static HRESULT WINAPI ddraw_surface4_IsLost(IDirectDrawSurface4 *iface) @@ -3693,7 +3695,7 @@ static HRESULT WINAPI ddraw_surface4_IsLost(IDirectDrawSurface4 *iface) TRACE("iface %p.\n", iface); - return ddraw_surface7_IsLost(&surface->IDirectDrawSurface7_iface); + return ddraw_surface_is_lost(surface) ? DDERR_SURFACELOST : DD_OK; } static HRESULT WINAPI ddraw_surface3_IsLost(IDirectDrawSurface3 *iface) @@ -3702,7 +3704,7 @@ static HRESULT WINAPI ddraw_surface3_IsLost(IDirectDrawSurface3 *iface) TRACE("iface %p.\n", iface); - return ddraw_surface7_IsLost(&surface->IDirectDrawSurface7_iface); + return ddraw_surface_is_lost(surface) ? DDERR_SURFACELOST : DD_OK; } static HRESULT WINAPI ddraw_surface2_IsLost(IDirectDrawSurface2 *iface) @@ -3711,7 +3713,7 @@ static HRESULT WINAPI ddraw_surface2_IsLost(IDirectDrawSurface2 *iface) TRACE("iface %p.\n", iface); - return ddraw_surface7_IsLost(&surface->IDirectDrawSurface7_iface); + return ddraw_surface_is_lost(surface) ? DDERR_SURFACELOST : DD_OK; } static HRESULT WINAPI ddraw_surface1_IsLost(IDirectDrawSurface *iface) @@ -3720,7 +3722,7 @@ static HRESULT WINAPI ddraw_surface1_IsLost(IDirectDrawSurface *iface) TRACE("iface %p.\n", iface); - return ddraw_surface7_IsLost(&surface->IDirectDrawSurface7_iface); + return ddraw_surface_is_lost(surface) ? DDERR_SURFACELOST : DD_OK; } /***************************************************************************** @@ -4774,7 +4776,7 @@ static HRESULT WINAPI ddraw_surface7_GetPalette(IDirectDrawSurface7 *iface, IDir if (!palette) return DDERR_INVALIDPARAMS; - if (IDirectDrawSurface7_IsLost(iface) == DDERR_SURFACELOST) + if (ddraw_surface_is_lost(surface)) { WARN("Surface lost, returning DDERR_SURFACELOST.\n"); return DDERR_SURFACELOST; @@ -4970,7 +4972,7 @@ static HRESULT WINAPI ddraw_surface7_SetPalette(IDirectDrawSurface7 *iface, IDir if (surface->surface_desc.ddsCaps.dwCaps2 & DDSCAPS2_MIPMAPSUBLEVEL) return DDERR_NOTONMIPMAPSUBLEVEL; - if (IDirectDrawSurface7_IsLost(iface) == DDERR_SURFACELOST) + if (ddraw_surface_is_lost(surface)) { WARN("Surface lost, returning DDERR_SURFACELOST.\n"); return DDERR_SURFACELOST; @@ -4985,7 +4987,7 @@ static HRESULT WINAPI ddraw_surface4_SetPalette(IDirectDrawSurface4 *iface, IDir TRACE("iface %p, palette %p.\n", iface, palette); - if (IDirectDrawSurface4_IsLost(iface) == DDERR_SURFACELOST) + if (ddraw_surface_is_lost(surface)) { WARN("Surface lost, returning DDERR_SURFACELOST.\n"); return DDERR_SURFACELOST; @@ -5000,7 +5002,7 @@ static HRESULT WINAPI ddraw_surface3_SetPalette(IDirectDrawSurface3 *iface, IDir TRACE("iface %p, palette %p.\n", iface, palette); - if (IDirectDrawSurface3_IsLost(iface) == DDERR_SURFACELOST) + if (ddraw_surface_is_lost(surface)) { WARN("Surface lost, returning DDERR_SURFACELOST.\n"); return DDERR_SURFACELOST; @@ -5015,7 +5017,7 @@ static HRESULT WINAPI ddraw_surface2_SetPalette(IDirectDrawSurface2 *iface, IDir TRACE("iface %p, palette %p.\n", iface, palette); - if (IDirectDrawSurface2_IsLost(iface) == DDERR_SURFACELOST) + if (ddraw_surface_is_lost(surface)) { WARN("Surface lost, returning DDERR_SURFACELOST.\n"); return DDERR_SURFACELOST; @@ -5030,7 +5032,7 @@ static HRESULT WINAPI ddraw_surface1_SetPalette(IDirectDrawSurface *iface, IDire TRACE("iface %p, palette %p.\n", iface, palette); - if (IDirectDrawSurface_IsLost(iface) == DDERR_SURFACELOST) + if (ddraw_surface_is_lost(surface)) { WARN("Surface lost, returning DDERR_SURFACELOST.\n"); return DDERR_SURFACELOST;

1 0