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

Stefan Dösinger : wined3d: Remove resource private data methods.
by Alexandre Julliard 24 Mar '14

24 Mar '14

Module: wine Branch: master Commit: ebd5f96aea22f4093d4ceb109bdead2657666c57 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ebd5f96aea22f4093d4ceb109… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Mon Mar 24 12:17:18 2014 +0100 wined3d: Remove resource private data methods. --- dlls/d3d9/volume.c | 30 +++------------------ dlls/wined3d/resource.c | 58 ---------------------------------------- dlls/wined3d/wined3d.spec | 3 --- dlls/wined3d/wined3d_private.h | 1 - include/wine/wined3d.h | 5 ---- 5 files changed, 3 insertions(+), 94 deletions(-) diff --git a/dlls/d3d9/volume.c b/dlls/d3d9/volume.c index a546c06..30a977c 100644 --- a/dlls/d3d9/volume.c +++ b/dlls/d3d9/volume.c @@ -123,52 +123,28 @@ static HRESULT WINAPI d3d9_volume_SetPrivateData(IDirect3DVolume9 *iface, REFGUI const void *data, DWORD data_size, DWORD flags) { struct d3d9_volume *volume = impl_from_IDirect3DVolume9(iface); - struct wined3d_resource *resource; - HRESULT hr; - TRACE("iface %p, guid %s, data %p, data_size %u, flags %#x.\n", iface, debugstr_guid(guid), data, data_size, flags); - wined3d_mutex_lock(); - resource = wined3d_volume_get_resource(volume->wined3d_volume); - hr = wined3d_resource_set_private_data(resource, guid, data, data_size, flags); - wined3d_mutex_unlock(); - - return hr; + return d3d9_resource_set_private_data(&volume->resource, guid, data, data_size, flags); } static HRESULT WINAPI d3d9_volume_GetPrivateData(IDirect3DVolume9 *iface, REFGUID guid, void *data, DWORD *data_size) { struct d3d9_volume *volume = impl_from_IDirect3DVolume9(iface); - struct wined3d_resource *resource; - HRESULT hr; - TRACE("iface %p, guid %s, data %p, data_size %p.\n", iface, debugstr_guid(guid), data, data_size); - wined3d_mutex_lock(); - resource = wined3d_volume_get_resource(volume->wined3d_volume); - hr = wined3d_resource_get_private_data(resource, guid, data, data_size); - wined3d_mutex_unlock(); - - return hr; + return d3d9_resource_get_private_data(&volume->resource, guid, data, data_size); } static HRESULT WINAPI d3d9_volume_FreePrivateData(IDirect3DVolume9 *iface, REFGUID guid) { struct d3d9_volume *volume = impl_from_IDirect3DVolume9(iface); - struct wined3d_resource *resource; - HRESULT hr; - TRACE("iface %p, guid %s.\n", iface, debugstr_guid(guid)); - wined3d_mutex_lock(); - resource = wined3d_volume_get_resource(volume->wined3d_volume); - hr = wined3d_resource_free_private_data(resource, guid); - wined3d_mutex_unlock(); - - return hr; + return d3d9_resource_free_private_data(&volume->resource, guid); } static HRESULT WINAPI d3d9_volume_GetContainer(IDirect3DVolume9 *iface, REFIID riid, void **container) diff --git a/dlls/wined3d/resource.c b/dlls/wined3d/resource.c index 43bd7ce..999dc5c 100644 --- a/dlls/wined3d/resource.c +++ b/dlls/wined3d/resource.c @@ -100,7 +100,6 @@ HRESULT resource_init(struct wined3d_resource *resource, struct wined3d_device * resource->parent = parent; resource->parent_ops = parent_ops; resource->resource_ops = resource_ops; - wined3d_private_store_init(&resource->private_store); if (size) { @@ -144,8 +143,6 @@ void resource_cleanup(struct wined3d_resource *resource) adapter_adjust_memory(resource->device->adapter, 0 - resource->size); } - wined3d_private_store_cleanup(&resource->private_store); - wined3d_resource_free_sysmem(resource); device_resource_released(resource->device, resource); @@ -160,61 +157,6 @@ void resource_unload(struct wined3d_resource *resource) resource, resource->type); } -HRESULT CDECL wined3d_resource_set_private_data(struct wined3d_resource *resource, REFGUID guid, - const void *data, DWORD data_size, DWORD flags) -{ - TRACE("resource %p, riid %s, data %p, data_size %u, flags %#x.\n", - resource, debugstr_guid(guid), data, data_size, flags); - - return wined3d_private_store_set_private_data(&resource->private_store, guid, data, data_size, flags); -} - -HRESULT CDECL wined3d_resource_get_private_data(const struct wined3d_resource *resource, REFGUID guid, - void *data, DWORD *data_size) -{ - const struct wined3d_private_data *d; - DWORD size_in; - - TRACE("resource %p, guid %s, data %p, data_size %p.\n", - resource, debugstr_guid(guid), data, data_size); - - d = wined3d_private_store_get_private_data(&resource->private_store, guid); - if (!d) - return WINED3DERR_NOTFOUND; - - size_in = *data_size; - *data_size = d->size; - if (!data) - return WINED3D_OK; - if (size_in < d->size) - return WINED3DERR_MOREDATA; - - if (d->flags & WINED3DSPD_IUNKNOWN) - { - *(IUnknown **)data = d->content.object; - IUnknown_AddRef(d->content.object); - } - else - { - memcpy(data, d->content.data, d->size); - } - - return WINED3D_OK; -} - -HRESULT CDECL wined3d_resource_free_private_data(struct wined3d_resource *resource, REFGUID guid) -{ - struct wined3d_private_data *entry; - TRACE("resource %p, guid %s.\n", resource, debugstr_guid(guid)); - - entry = wined3d_private_store_get_private_data(&resource->private_store, guid); - if (!entry) - return WINED3DERR_NOTFOUND; - wined3d_private_store_free_private_data(&resource->private_store, entry); - - return WINED3D_OK; -} - DWORD resource_set_priority(struct wined3d_resource *resource, DWORD priority) { DWORD prev = resource->priority; diff --git a/dlls/wined3d/wined3d.spec b/dlls/wined3d/wined3d.spec index 3243f77..bdfe017 100644 --- a/dlls/wined3d/wined3d.spec +++ b/dlls/wined3d/wined3d.spec @@ -170,12 +170,9 @@ @ cdecl wined3d_query_incref(ptr) @ cdecl wined3d_query_issue(ptr long) -@ cdecl wined3d_resource_free_private_data(ptr ptr) @ cdecl wined3d_resource_get_desc(ptr ptr) @ cdecl wined3d_resource_get_parent(ptr) -@ cdecl wined3d_resource_get_private_data(ptr ptr ptr ptr) @ cdecl wined3d_resource_set_parent(ptr ptr) -@ cdecl wined3d_resource_set_private_data(ptr ptr ptr long long) @ cdecl wined3d_rendertarget_view_create(ptr ptr ptr) @ cdecl wined3d_rendertarget_view_decref(ptr) diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 7abcc8a..9e209d9 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -2008,7 +2008,6 @@ struct wined3d_resource DWORD priority; void *heap_memory; struct list resource_list_entry; - struct wined3d_private_store private_store; void *parent; const struct wined3d_parent_ops *parent_ops; diff --git a/include/wine/wined3d.h b/include/wine/wined3d.h index e6fa2d1..c958872 100644 --- a/include/wine/wined3d.h +++ b/include/wine/wined3d.h @@ -2340,15 +2340,10 @@ static inline HRESULT wined3d_private_store_set_private_data(struct wined3d_priv return WINED3D_OK; } -HRESULT __cdecl wined3d_resource_free_private_data(struct wined3d_resource *resource, REFGUID guid); void __cdecl wined3d_resource_get_desc(const struct wined3d_resource *resource, struct wined3d_resource_desc *desc); void * __cdecl wined3d_resource_get_parent(const struct wined3d_resource *resource); -HRESULT __cdecl wined3d_resource_get_private_data(const struct wined3d_resource *resource, - REFGUID guid, void *data, DWORD *data_size); void __cdecl wined3d_resource_set_parent(struct wined3d_resource *resource, void *parent); -HRESULT __cdecl wined3d_resource_set_private_data(struct wined3d_resource *resource, - REFGUID guid, const void *data, DWORD data_size, DWORD flags); HRESULT __cdecl wined3d_rendertarget_view_create(struct wined3d_resource *resource, void *parent, struct wined3d_rendertarget_view **rendertarget_view);

1 0

Nikolay Sivov : oleaut32/tests: Move string allocation after function pointer check (Coverity).
by Alexandre Julliard 24 Mar '14

24 Mar '14

Module: wine Branch: master Commit: e5e86776664a19aeb009fb27f5fe6efcf04494ba URL: http://source.winehq.org/git/wine.git/?a=commit;h=e5e86776664a19aeb009fb27f… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Mon Mar 24 20:01:50 2014 +0400 oleaut32/tests: Move string allocation after function pointer check (Coverity). --- dlls/oleaut32/tests/vartest.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/dlls/oleaut32/tests/vartest.c b/dlls/oleaut32/tests/vartest.c index d28b5c1..bfbc0b2 100644 --- a/dlls/oleaut32/tests/vartest.c +++ b/dlls/oleaut32/tests/vartest.c @@ -6000,11 +6000,11 @@ static void test_VarAnd(void) VARTYPE i; HRESULT hres; + CHECKPTR(VarAnd); + true_str = SysAllocString(szTrue); false_str = SysAllocString(szFalse); - CHECKPTR(VarAnd); - /* Test all possible flag/vt combinations & the resulting vt type */ for (i = 0; i < sizeof(ExtraFlags)/sizeof(ExtraFlags[0]); i++) { @@ -7479,11 +7479,11 @@ static void test_VarDiv(void) HRESULT hres; double r; + CHECKPTR(VarDiv); + num1_str = SysAllocString(str1); num2_str = SysAllocString(str2); - CHECKPTR(VarDiv); - /* Test all possible flag/vt combinations & the resulting vt type */ for (i = 0; i < sizeof(ExtraFlags)/sizeof(ExtraFlags[0]); i++) {

1 0

Ken Thomases : wined3d: Don' t leak the HDC of an invalid context in context_update_window().
by Alexandre Julliard 24 Mar '14

24 Mar '14

Module: wine Branch: master Commit: 03ca2b2932b38e0b79592787e32013c1e5ae66bd URL: http://source.winehq.org/git/wine.git/?a=commit;h=03ca2b2932b38e0b79592787e… Author: Ken Thomases <ken(a)codeweavers.com> Date: Sun Mar 23 22:45:44 2014 -0500 wined3d: Don't leak the HDC of an invalid context in context_update_window(). If context_set_gl_context() fails, it marks the context as invalid. It still has an HDC, though, and context_update_window() would lose the reference to it. --- dlls/wined3d/context.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index 56673c7..73e2c39 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -884,13 +884,12 @@ static void context_update_window(struct wined3d_context *context) TRACE("Updating context %p window from %p to %p.\n", context, context->win_handle, context->swapchain->win_handle); - if (context->valid) + if (context->hdc) wined3d_release_dc(context->win_handle, context->hdc); - else - context->valid = 1; context->win_handle = context->swapchain->win_handle; context->needs_set = 1; + context->valid = 1; if (!(context->hdc = GetDC(context->win_handle))) {

1 0

Ken Thomases : wined3d: Simplify the error path of context_update_window() .
by Alexandre Julliard 24 Mar '14

24 Mar '14

Module: wine Branch: master Commit: 90d7f3e03f584f15f94fa7c75b51342f725d2f1e URL: http://source.winehq.org/git/wine.git/?a=commit;h=90d7f3e03f584f15f94fa7c75… Author: Ken Thomases <ken(a)codeweavers.com> Date: Sun Mar 23 22:45:50 2014 -0500 wined3d: Simplify the error path of context_update_window(). --- dlls/wined3d/context.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index 73e2c39..f90557e 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -894,13 +894,8 @@ static void context_update_window(struct wined3d_context *context) if (!(context->hdc = GetDC(context->win_handle))) { ERR("Failed to get a device context for window %p.\n", context->win_handle); - goto err; + context->valid = 0; } - - return; - -err: - context->valid = 0; } static void context_destroy_gl_resources(struct wined3d_context *context)

1 0

Ken Thomases : winemac: Improve handling of swap interval with respect to pbuffers, clamping to 0.
by Alexandre Julliard 24 Mar '14

24 Mar '14

Module: wine Branch: master Commit: fc37ed94d85fb951aad9c94fa3b3ac946f4670ae URL: http://source.winehq.org/git/wine.git/?a=commit;h=fc37ed94d85fb951aad9c94fa… Author: Ken Thomases <ken(a)codeweavers.com> Date: Sun Mar 23 22:44:25 2014 -0500 winemac: Improve handling of swap interval with respect to pbuffers, clamping to 0. --- dlls/winemac.drv/opengl.c | 66 ++++++++++++++++++++++++++++++--------------- 1 file changed, 44 insertions(+), 22 deletions(-) diff --git a/dlls/winemac.drv/opengl.c b/dlls/winemac.drv/opengl.c index 4265797..0187822 100644 --- a/dlls/winemac.drv/opengl.c +++ b/dlls/winemac.drv/opengl.c @@ -1504,13 +1504,25 @@ static BOOL set_swap_interval(struct wgl_context *context, long interval) */ static void sync_swap_interval(struct wgl_context *context) { - struct macdrv_win_data *data; - - if (InterlockedCompareExchange(&context->update_swap_interval, FALSE, TRUE) && - (data = get_win_data(context->draw_hwnd))) + if (InterlockedCompareExchange(&context->update_swap_interval, FALSE, TRUE)) { - set_swap_interval(context, data->swap_interval); - release_win_data(data); + int interval; + + if (context->draw_hwnd) + { + struct macdrv_win_data *data = get_win_data(context->draw_hwnd); + if (data) + { + interval = data->swap_interval; + release_win_data(data); + } + else /* window was destroyed? */ + interval = 1; + } + else /* pbuffer */ + interval = 0; + + set_swap_interval(context, interval); } } @@ -2718,6 +2730,10 @@ static BOOL macdrv_wglMakeContextCurrentARB(HDC draw_hdc, HDC read_hdc, struct w SetLastError(ERROR_INVALID_PIXEL_FORMAT); return FALSE; } + + if (allow_vsync && + (InterlockedCompareExchange(&context->update_swap_interval, FALSE, TRUE) || pbuffer != context->draw_pbuffer)) + set_swap_interval(context, 0); } else { @@ -3019,7 +3035,7 @@ static BOOL macdrv_wglSetPixelFormatWINE(HDC hdc, int fmt) static BOOL macdrv_wglSwapIntervalEXT(int interval) { struct wgl_context *context = NtCurrentTeb()->glContext; - struct macdrv_win_data *data; + BOOL changed = FALSE; TRACE("interval %d\n", interval); @@ -3031,6 +3047,20 @@ static BOOL macdrv_wglSwapIntervalEXT(int interval) if (interval > 1) interval = 1; + if (context->draw_hwnd) + { + struct macdrv_win_data *data = get_win_data(context->draw_hwnd); + if (data) + { + changed = data->swap_interval != interval; + if (changed) + data->swap_interval = interval; + release_win_data(data); + } + } + else /* pbuffer */ + interval = 0; + InterlockedExchange(&context->update_swap_interval, FALSE); if (!set_swap_interval(context, interval)) { @@ -3038,25 +3068,17 @@ static BOOL macdrv_wglSwapIntervalEXT(int interval) return FALSE; } - if ((data = get_win_data(context->draw_hwnd))) + if (changed) { - BOOL changed = data->swap_interval != interval; - if (changed) - data->swap_interval = interval; - release_win_data(data); + struct wgl_context *ctx; - if (changed) + EnterCriticalSection(&context_section); + LIST_FOR_EACH_ENTRY(ctx, &context_list, struct wgl_context, entry) { - struct wgl_context *ctx; - - EnterCriticalSection(&context_section); - LIST_FOR_EACH_ENTRY(ctx, &context_list, struct wgl_context, entry) - { - if (ctx != context && ctx->draw_hwnd == context->draw_hwnd) - InterlockedExchange(&context->update_swap_interval, TRUE); - } - LeaveCriticalSection(&context_section); + if (ctx != context && ctx->draw_hwnd == context->draw_hwnd) + InterlockedExchange(&context->update_swap_interval, TRUE); } + LeaveCriticalSection(&context_section); } return TRUE;

1 0

Dmitry Timoshkov : include: Fix a typo in _MSC_VER.
by Alexandre Julliard 24 Mar '14

24 Mar '14

Module: wine Branch: master Commit: 4dff2e7157f7c1c712eb98ec3f813fe2b5b80e87 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4dff2e7157f7c1c712eb98ec3… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Mar 24 16:08:16 2014 +0900 include: Fix a typo in _MSC_VER. --- include/rpcndr.h | 2 +- include/winnt.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/rpcndr.h b/include/rpcndr.h index 99200ac..f795566 100644 --- a/include/rpcndr.h +++ b/include/rpcndr.h @@ -429,7 +429,7 @@ typedef struct _MIDL_STUBLESS_PROXY_INFO } MIDL_STUBLESS_PROXY_INFO, *PMIDL_STUBLESS_PROXY_INFO; -#if defined(__i386__) && !defined(__MSC_VER) && !defined(__MINGW32__) && !defined(__CYGWIN__) +#if defined(__i386__) && !defined(_MSC_VER) && !defined(__MINGW32__) && !defined(__CYGWIN__) /* Calling convention for returning structures/unions is different between Windows and gcc on i386 */ typedef LONG_PTR CLIENT_CALL_RETURN; #else diff --git a/include/winnt.h b/include/winnt.h index f785d33..93fd70a 100644 --- a/include/winnt.h +++ b/include/winnt.h @@ -154,7 +154,7 @@ extern "C" { # define DECLSPEC_EXPORT #endif -#if defined(__MSC_VER) || defined(__MINGW32__) || defined(__CYGWIN__) +#if defined(_MSC_VER) || defined(__MINGW32__) || defined(__CYGWIN__) # define DECLSPEC_HIDDEN #elif defined(__GNUC__) && ((__GNUC__ > 3) || ((__GNUC__ == 3) && (__GNUC_MINOR__ >= 3))) # define DECLSPEC_HIDDEN __attribute__((visibility ("hidden")))

1 0

Dmitry Timoshkov : schedsvc: Skip the UTF-8 byte order mark when returing XML to the client.
by Alexandre Julliard 24 Mar '14

24 Mar '14

Module: wine Branch: master Commit: 285922ca7115424bb4bf3ce5b5bda7ffba8eba96 URL: http://source.winehq.org/git/wine.git/?a=commit;h=285922ca7115424bb4bf3ce5b… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Mar 24 16:08:34 2014 +0900 schedsvc: Skip the UTF-8 byte order mark when returing XML to the client. --- dlls/schedsvc/schedsvc.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/dlls/schedsvc/schedsvc.c b/dlls/schedsvc/schedsvc.c index 988430d..5c7d359 100644 --- a/dlls/schedsvc/schedsvc.c +++ b/dlls/schedsvc/schedsvc.c @@ -29,6 +29,8 @@ WINE_DEFAULT_DEBUG_CHANNEL(schedsvc); +static const char bom_utf8[] = { 0xef,0xbb,0xbf }; + HRESULT __cdecl SchRpcHighestVersion(DWORD *version) { TRACE("%p\n", version); @@ -105,7 +107,6 @@ static HRESULT create_directory(const WCHAR *path) static HRESULT write_xml_utf8(const WCHAR *name, DWORD disposition, const WCHAR *xmlW) { - static const char bom_utf8[] = { 0xef,0xbb,0xbf }; static const char comment[] = "\n"; HANDLE hfile; DWORD size; @@ -220,8 +221,6 @@ HRESULT __cdecl SchRpcRegisterTask(const WCHAR *path, const WCHAR *xml, DWORD fl static int detect_encoding(const void *buffer, DWORD size) { - static const char bom_utf8[] = { 0xef,0xbb,0xbf }; - if (size >= sizeof(bom_utf8) && !memcmp(buffer, bom_utf8, sizeof(bom_utf8))) return CP_UTF8; else @@ -270,6 +269,9 @@ static HRESULT read_xml(const WCHAR *name, WCHAR **xml) return S_OK; } + if (cp == CP_UTF8 && size >= sizeof(bom_utf8) && !memcmp(src, bom_utf8, sizeof(bom_utf8))) + src += sizeof(bom_utf8); + size = MultiByteToWideChar(cp, 0, src, -1, NULL, 0); *xml = heap_alloc(size * sizeof(WCHAR)); if (!*xml) return E_OUTOFMEMORY;

1 0

Henri Verbeet : d3d9/tests: Use a separate device for texdepth_test().
by Alexandre Julliard 24 Mar '14

24 Mar '14

Module: wine Branch: master Commit: 65c9b48de6f6b1c47089413bea8edcf0846169d8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=65c9b48de6f6b1c47089413be… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Mar 24 11:31:33 2014 +0100 d3d9/tests: Use a separate device for texdepth_test(). --- dlls/d3d9/tests/visual.c | 71 +++++++++++++++++++++++++++++++--------------- 1 file changed, 48 insertions(+), 23 deletions(-) diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index 9d3d201..1d27326 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -4679,17 +4679,24 @@ static void texture_transform_flags_test(IDirect3DDevice9 *device) IDirect3DVertexDeclaration9_Release(decl4); } -static void texdepth_test(IDirect3DDevice9 *device) +static void texdepth_test(void) { IDirect3DPixelShader9 *shader; + IDirect3DDevice9 *device; + IDirect3D9 *d3d; + ULONG refcount; + D3DCAPS9 caps; + DWORD color; + HWND window; HRESULT hr; - const float texdepth_test_data1[] = { 0.25, 2.0, 0.0, 0.0}; - const float texdepth_test_data2[] = { 0.25, 0.5, 0.0, 0.0}; - const float texdepth_test_data3[] = {-1.00, 0.1, 0.0, 0.0}; - const float texdepth_test_data4[] = {-0.25, -0.5, 0.0, 0.0}; - const float texdepth_test_data5[] = { 1.00, -0.1, 0.0, 0.0}; - const float texdepth_test_data6[] = { 1.00, 0.5, 0.0, 0.0}; - const float texdepth_test_data7[] = { 0.50, 0.0, 0.0, 0.0}; + + static const float texdepth_test_data1[] = { 0.25f, 2.0f, 0.0f, 0.0f}; + static const float texdepth_test_data2[] = { 0.25f, 0.5f, 0.0f, 0.0f}; + static const float texdepth_test_data3[] = {-1.00f, 0.1f, 0.0f, 0.0f}; + static const float texdepth_test_data4[] = {-0.25f, -0.5f, 0.0f, 0.0f}; + static const float texdepth_test_data5[] = { 1.00f, -0.1f, 0.0f, 0.0f}; + static const float texdepth_test_data6[] = { 1.00f, 0.5f, 0.0f, 0.0f}; + static const float texdepth_test_data7[] = { 0.50f, 0.0f, 0.0f, 0.0f}; static const DWORD shader_code[] = { 0xffff0104, /* ps_1_4 */ @@ -4700,19 +4707,40 @@ static void texdepth_test(IDirect3DDevice9 *device) 0x00000001, 0x800f0000, 0xa0e40001, /* mov r0, c1 */ 0x0000ffff /* end */ }; - DWORD color; - float vertex[] = { - -1.0, -1.0, 0.0, - 1.0, -1.0, 1.0, - -1.0, 1.0, 0.0, - 1.0, 1.0, 1.0 + static const float vertex[] = + { + -1.0f, -1.0f, 0.0f, + -1.0f, 1.0f, 0.0f, + 1.0f, -1.0f, 1.0f, + 1.0f, 1.0f, 1.0f, }; + window = CreateWindowA("static", "d3d9_test", WS_OVERLAPPEDWINDOW | WS_VISIBLE, + 0, 0, 640, 480, NULL, NULL, NULL, NULL); + d3d = Direct3DCreate9(D3D_SDK_VERSION); + ok(!!d3d, "Failed to create a D3D object.\n"); + if (!(device = create_device(d3d, window, window, TRUE))) + { + skip("Failed to create a D3D device, skipping tests.\n"); + goto done; + } + + hr = IDirect3DDevice9_GetDeviceCaps(device, &caps); + ok(SUCCEEDED(hr), "Failed to get device caps, hr %#x.\n", hr); + if (caps.PixelShaderVersion < D3DPS_VERSION(1, 1)) + { + skip("No ps_1_1 support, skipping tests.\n"); + IDirect3DDevice9_Release(device); + goto done; + } + hr = IDirect3DDevice9_CreatePixelShader(device, shader_code, &shader); ok(hr == D3D_OK, "IDirect3DDevice9_CreatePixelShader returned %08x\n", hr); hr = IDirect3DDevice9_Clear(device, 0, NULL, D3DCLEAR_TARGET | D3DCLEAR_ZBUFFER, 0xffffff00, 0.0, 0); ok(hr == D3D_OK, "IDirect3DDevice9_Clear returned %08x\n", hr); + hr = IDirect3DDevice9_SetRenderState(device, D3DRS_LIGHTING, FALSE); + ok(SUCCEEDED(hr), "Failed to disable lighting, hr %#x.\n", hr); hr = IDirect3DDevice9_SetRenderState(device, D3DRS_ZENABLE, D3DZB_TRUE); ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState returned %08x\n", hr); hr = IDirect3DDevice9_SetRenderState(device, D3DRS_ZWRITEENABLE, TRUE); @@ -4897,15 +4925,12 @@ static void texdepth_test(IDirect3DDevice9 *device) hr = IDirect3DDevice9_Present(device, NULL, NULL, NULL, NULL); ok(hr == D3D_OK, "IDirect3DDevice9_Present failed with %08x\n", hr); - /* Cleanup */ - hr = IDirect3DDevice9_SetPixelShader(device, NULL); - ok(hr == D3D_OK, "IDirect3DDevice9_SetPixelShader failed (%08x)\n", hr); IDirect3DPixelShader9_Release(shader); - - hr = IDirect3DDevice9_SetRenderState(device, D3DRS_ZENABLE, D3DZB_FALSE); - ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState returned %08x\n", hr); - hr = IDirect3DDevice9_SetRenderState(device, D3DRS_ZWRITEENABLE, TRUE); - ok(hr == D3D_OK, "IDirect3DDevice9_SetRenderState returned %08x\n", hr); + refcount = IDirect3DDevice9_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); +done: + IDirect3D9_Release(d3d); + DestroyWindow(window); } static void texkill_test(void) @@ -16043,13 +16068,13 @@ START_TEST(visual) if (caps.PixelShaderVersion >= D3DPS_VERSION(1, 1)) { texbem_test(device_ptr); - texdepth_test(device_ptr); } else skip("No ps_1_1 support\n"); cleanup_device(device_ptr); device_ptr = NULL; + texdepth_test(); texkill_test(); x8l8v8u8_test(); volume_v16u16_test();

1 0

Henri Verbeet : d3d9/tests: Use a separate device for texbem_test().
by Alexandre Julliard 24 Mar '14

24 Mar '14

Module: wine Branch: master Commit: aded2bb23ca1d745a32771d91f74ebe442c16ee3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=aded2bb23ca1d745a32771d91… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Mar 24 11:31:34 2014 +0100 d3d9/tests: Use a separate device for texbem_test(). --- dlls/d3d9/tests/visual.c | 136 +++++++++++++++++++++++++--------------------- 1 file changed, 75 insertions(+), 61 deletions(-) diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index 1d27326..c736bfb 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -2179,23 +2179,36 @@ static void generate_bumpmap_textures(IDirect3DDevice9 *device) { } } -/* test the behavior of the texbem instruction - * with normal 2D and projective 2D textures - */ -static void texbem_test(IDirect3DDevice9 *device) +/* Test the behavior of the texbem instruction with normal 2D and projective + * 2D textures. */ +static void texbem_test(void) { - HRESULT hr; + IDirect3DVertexDeclaration9 *vertex_declaration = NULL; + /* Use asymmetric matrix to test loading. */ + float bumpenvmat[4] = {0.0f, 0.5f, -0.5f, 0.0f}; + IDirect3DPixelShader9 *pixel_shader = NULL; + IDirect3DTexture9 *texture1, *texture2; + IDirect3DTexture9 *texture = NULL; + D3DLOCKED_RECT locked_rect; + IDirect3DDevice9 *device; + IDirect3D9 *d3d; + ULONG refcount; + D3DCAPS9 caps; DWORD color; + HWND window; + HRESULT hr; int i; - static const DWORD pixel_shader_code[] = { + static const DWORD pixel_shader_code[] = + { 0xffff0101, /* ps_1_1*/ 0x00000042, 0xb00f0000, /* tex t0*/ 0x00000043, 0xb00f0001, 0xb0e40000, /* texbem t1, t0*/ 0x00000001, 0x800f0000, 0xb0e40001, /* mov r0, t1*/ 0x0000ffff }; - static const DWORD double_texbem_code[] = { + static const DWORD double_texbem_code[] = + { 0xffff0103, /* ps_1_3 */ 0x00000042, 0xb00f0000, /* tex t0 */ 0x00000043, 0xb00f0001, 0xb0e40000, /* texbem t1, t0 */ @@ -2204,40 +2217,61 @@ static void texbem_test(IDirect3DDevice9 *device) 0x00000002, 0x800f0000, 0xb0e40001, 0xb0e40003, /* add r0, t1, t3 */ 0x0000ffff /* end */ }; - - - static const float quad[][7] = { + static const float quad[][7] = + { {-128.0f/640.0f, -128.0f/480.0f, 0.1f, 0.0f, 0.0f, 0.0f, 0.0f}, {-128.0f/640.0f, 128.0f/480.0f, 0.1f, 0.0f, 1.0f, 0.0f, 1.0f}, { 128.0f/640.0f, -128.0f/480.0f, 0.1f, 1.0f, 0.0f, 1.0f, 0.0f}, { 128.0f/640.0f, 128.0f/480.0f, 0.1f, 1.0f, 1.0f, 1.0f, 1.0f}, }; - static const float quad_proj[][9] = { + static const float quad_proj[][9] = + { {-128.0f/640.0f, -128.0f/480.0f, 0.1f, 0.0f, 0.0f, 0.0f, 0.0f, 0.0f, 128.0f}, {-128.0f/640.0f, 128.0f/480.0f, 0.1f, 0.0f, 1.0f, 0.0f, 128.0f, 0.0f, 128.0f}, { 128.0f/640.0f, -128.0f/480.0f, 0.1f, 1.0f, 0.0f, 128.0f, 0.0f, 0.0f, 128.0f}, { 128.0f/640.0f, 128.0f/480.0f, 0.1f, 1.0f, 1.0f, 128.0f, 128.0f, 0.0f, 128.0f}, }; + static const float double_quad[] = + { + -1.0f, -1.0f, 0.0f, 0.0f, 0.0f, 0.5f, 0.5f, 0.0f, 0.0f, 0.5f, 0.5f, + -1.0f, 1.0f, 0.0f, 0.0f, 0.0f, 0.5f, 0.5f, 0.0f, 0.0f, 0.5f, 0.5f, + 1.0f, -1.0f, 0.0f, 0.0f, 0.0f, 0.5f, 0.5f, 0.0f, 0.0f, 0.5f, 0.5f, + 1.0f, 1.0f, 0.0f, 0.0f, 0.0f, 0.5f, 0.5f, 0.0f, 0.0f, 0.5f, 0.5f, + }; + static const D3DVERTEXELEMENT9 decl_elements[][4] = + { + { + {0, 0, D3DDECLTYPE_FLOAT3, D3DDECLMETHOD_DEFAULT, D3DDECLUSAGE_POSITION, 0}, + {0, 12, D3DDECLTYPE_FLOAT2, D3DDECLMETHOD_DEFAULT, D3DDECLUSAGE_TEXCOORD, 0}, + {0, 20, D3DDECLTYPE_FLOAT2, D3DDECLMETHOD_DEFAULT, D3DDECLUSAGE_TEXCOORD, 1}, + D3DDECL_END() + }, + { + {0, 0, D3DDECLTYPE_FLOAT3, D3DDECLMETHOD_DEFAULT, D3DDECLUSAGE_POSITION, 0}, + {0, 12, D3DDECLTYPE_FLOAT2, D3DDECLMETHOD_DEFAULT, D3DDECLUSAGE_TEXCOORD, 0}, + {0, 20, D3DDECLTYPE_FLOAT4, D3DDECLMETHOD_DEFAULT, D3DDECLUSAGE_TEXCOORD, 1}, + D3DDECL_END() + }, + }; - static const D3DVERTEXELEMENT9 decl_elements[][4] = { { - {0, 0, D3DDECLTYPE_FLOAT3, D3DDECLMETHOD_DEFAULT, D3DDECLUSAGE_POSITION, 0}, - {0, 12, D3DDECLTYPE_FLOAT2, D3DDECLMETHOD_DEFAULT, D3DDECLUSAGE_TEXCOORD, 0}, - {0, 20, D3DDECLTYPE_FLOAT2, D3DDECLMETHOD_DEFAULT, D3DDECLUSAGE_TEXCOORD, 1}, - D3DDECL_END() - },{ - {0, 0, D3DDECLTYPE_FLOAT3, D3DDECLMETHOD_DEFAULT, D3DDECLUSAGE_POSITION, 0}, - {0, 12, D3DDECLTYPE_FLOAT2, D3DDECLMETHOD_DEFAULT, D3DDECLUSAGE_TEXCOORD, 0}, - {0, 20, D3DDECLTYPE_FLOAT4, D3DDECLMETHOD_DEFAULT, D3DDECLUSAGE_TEXCOORD, 1}, - D3DDECL_END() - } }; - - /* use asymmetric matrix to test loading */ - float bumpenvmat[4] = {0.0,0.5,-0.5,0.0}; + window = CreateWindowA("static", "d3d9_test", WS_OVERLAPPEDWINDOW | WS_VISIBLE, + 0, 0, 640, 480, NULL, NULL, NULL, NULL); + d3d = Direct3DCreate9(D3D_SDK_VERSION); + ok(!!d3d, "Failed to create a D3D object.\n"); + if (!(device = create_device(d3d, window, window, TRUE))) + { + skip("Failed to create a D3D device, skipping tests.\n"); + goto done; + } - IDirect3DVertexDeclaration9 *vertex_declaration = NULL; - IDirect3DPixelShader9 *pixel_shader = NULL; - IDirect3DTexture9 *texture = NULL, *texture1, *texture2; - D3DLOCKED_RECT locked_rect; + hr = IDirect3DDevice9_GetDeviceCaps(device, &caps); + ok(SUCCEEDED(hr), "Failed to get device caps, hr %#x.\n", hr); + if (caps.PixelShaderVersion < D3DPS_VERSION(1, 1)) + { + skip("No ps_1_1 support, skipping tests.\n"); + IDirect3DDevice9_Release(device); + goto done; + } generate_bumpmap_textures(device); @@ -2250,7 +2284,7 @@ static void texbem_test(IDirect3DDevice9 *device) hr = IDirect3DDevice9_SetVertexShader(device, NULL); ok(SUCCEEDED(hr), "SetVertexShader failed (%08x)\n", hr); - hr = IDirect3DDevice9_Clear(device, 0, NULL, D3DCLEAR_TARGET, 0xffff00ff, 0.0, 0); + hr = IDirect3DDevice9_Clear(device, 0, NULL, D3DCLEAR_TARGET | D3DCLEAR_ZBUFFER, 0xffff00ff, 1.0f, 0); ok(hr == D3D_OK, "IDirect3DDevice9_Clear failed (%08x)\n", hr); for(i=0; i<2; i++) @@ -2427,34 +2461,14 @@ static void texbem_test(IDirect3DDevice9 *device) ok(SUCCEEDED(hr), "IDirect3DDevice9_SetSamplerState returned %#x.\n", hr); hr = IDirect3DDevice9_BeginScene(device); - ok(SUCCEEDED(hr), "BeginScene failed (0x%08x)\n", hr); - if(SUCCEEDED(hr)) { - static const float double_quad[] = { - -1.0, -1.0, 0.0, 0.0, 0.0, 0.5, 0.5, 0.0, 0.0, 0.5, 0.5, - 1.0, -1.0, 0.0, 0.0, 0.0, 0.5, 0.5, 0.0, 0.0, 0.5, 0.5, - -1.0, 1.0, 0.0, 0.0, 0.0, 0.5, 0.5, 0.0, 0.0, 0.5, 0.5, - 1.0, 1.0, 0.0, 0.0, 0.0, 0.5, 0.5, 0.0, 0.0, 0.5, 0.5, - }; - - hr = IDirect3DDevice9_DrawPrimitiveUP(device, D3DPT_TRIANGLESTRIP, 2, double_quad, sizeof(float) * 11); - ok(SUCCEEDED(hr), "DrawPrimitiveUP failed (0x%08x)\n", hr); - hr = IDirect3DDevice9_EndScene(device); - ok(SUCCEEDED(hr), "EndScene failed (0x%08x)\n", hr); - } + ok(SUCCEEDED(hr), "Failed to begin scene, hr %#x.\n", hr); + hr = IDirect3DDevice9_DrawPrimitiveUP(device, D3DPT_TRIANGLESTRIP, 2, double_quad, sizeof(float) * 11); + ok(SUCCEEDED(hr), "Failed to draw primitive, hr %#x.\n", hr); + hr = IDirect3DDevice9_EndScene(device); + ok(SUCCEEDED(hr), "Failed to end scene, hr %#x.\n", hr); color = getPixelColor(device, 320, 240); ok(color == 0x00ffff00, "double texbem failed: Got color 0x%08x, expected 0x00ffff00.\n", color); - hr = IDirect3DDevice9_SetTexture(device, 0, NULL); - ok(SUCCEEDED(hr), "IDirect3DDevice9_SetTexture failed (0x%08x)\n", hr); - hr = IDirect3DDevice9_SetTexture(device, 1, NULL); - ok(SUCCEEDED(hr), "IDirect3DDevice9_SetTexture failed (0x%08x)\n", hr); - hr = IDirect3DDevice9_SetTexture(device, 2, NULL); - ok(SUCCEEDED(hr), "IDirect3DDevice9_SetTexture failed (0x%08x)\n", hr); - hr = IDirect3DDevice9_SetTexture(device, 3, NULL); - ok(SUCCEEDED(hr), "IDirect3DDevice9_SetTexture failed (0x%08x)\n", hr); - hr = IDirect3DDevice9_SetPixelShader(device, NULL); - ok(SUCCEEDED(hr), "Direct3DDevice9_SetPixelShader failed (0x%08x)\n", hr); - hr = IDirect3DDevice9_Present(device, NULL, NULL, NULL, NULL); ok(SUCCEEDED(hr), "Present failed (0x%08x)\n", hr); @@ -2462,6 +2476,11 @@ static void texbem_test(IDirect3DDevice9 *device) IDirect3DTexture9_Release(texture); IDirect3DTexture9_Release(texture1); IDirect3DTexture9_Release(texture2); + refcount = IDirect3DDevice9_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); +done: + IDirect3D9_Release(d3d); + DestroyWindow(window); } static void z_range_test(IDirect3DDevice9 *device) @@ -16065,15 +16084,10 @@ START_TEST(visual) } else skip("No vs_2_0 and ps_2_0 support\n"); - if (caps.PixelShaderVersion >= D3DPS_VERSION(1, 1)) - { - texbem_test(device_ptr); - } - else skip("No ps_1_1 support\n"); - cleanup_device(device_ptr); device_ptr = NULL; + texbem_test(); texdepth_test(); texkill_test(); x8l8v8u8_test();

1 0

Henri Verbeet : d3d9/tests: Use a separate device for fog_with_shader_test ().
by Alexandre Julliard 24 Mar '14

24 Mar '14

Module: wine Branch: master Commit: 0be10d60503549a2ae25d7efa4a63e946782226b URL: http://source.winehq.org/git/wine.git/?a=commit;h=0be10d60503549a2ae25d7efa… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Mar 24 11:31:35 2014 +0100 d3d9/tests: Use a separate device for fog_with_shader_test(). --- dlls/d3d9/tests/visual.c | 106 +++++++++++++++++++++++++++++----------------- 1 file changed, 68 insertions(+), 38 deletions(-) diff --git a/dlls/d3d9/tests/visual.c b/dlls/d3d9/tests/visual.c index c736bfb..29649d8 100644 --- a/dlls/d3d9/tests/visual.c +++ b/dlls/d3d9/tests/visual.c @@ -1746,17 +1746,25 @@ out: * linear table fog with non foggy vertex shader * vertex fog with foggy vertex shader, non-linear * fog with shader, non-linear fog with foggy shader, - * linear table fog with foggy shader - */ -static void fog_with_shader_test(IDirect3DDevice9 *device) + * linear table fog with foggy shader */ +static void fog_with_shader_test(void) { - HRESULT hr; + IDirect3DVertexShader9 *vertex_shader[4] = {NULL, NULL, NULL, NULL}; + IDirect3DPixelShader9 *pixel_shader[3] = {NULL, NULL, NULL}; + IDirect3DVertexDeclaration9 *vertex_declaration = NULL; + IDirect3DDevice9 *device; + unsigned int i, j; + IDirect3D9 *d3d; + ULONG refcount; + D3DCAPS9 caps; DWORD color; - union { + HWND window; + HRESULT hr; + union + { float f; DWORD i; } start, end; - unsigned int i, j; /* basic vertex shader without fog computation ("non foggy") */ static const DWORD vertex_shader_code1[] = @@ -1808,32 +1816,31 @@ static void fog_with_shader_test(IDirect3DDevice9 *device) 0x02000001, 0x800f0800, 0x90e40000, /* mov oC0, v0 */ 0x0000ffff }; - - static struct vertex quad[] = { + static struct vertex quad[] = + { {-1.0f, -1.0f, 0.0f, 0xffff0000 }, {-1.0f, 1.0f, 0.0f, 0xffff0000 }, { 1.0f, -1.0f, 0.0f, 0xffff0000 }, { 1.0f, 1.0f, 0.0f, 0xffff0000 }, }; - - static const D3DVERTEXELEMENT9 decl_elements[] = { + static const D3DVERTEXELEMENT9 decl_elements[] = + { {0, 0, D3DDECLTYPE_FLOAT3, D3DDECLMETHOD_DEFAULT, D3DDECLUSAGE_POSITION, 0}, {0, 12, D3DDECLTYPE_D3DCOLOR, D3DDECLMETHOD_DEFAULT, D3DDECLUSAGE_COLOR, 0}, D3DDECL_END() }; - - IDirect3DVertexDeclaration9 *vertex_declaration = NULL; - IDirect3DVertexShader9 *vertex_shader[4] = {NULL, NULL, NULL, NULL}; - IDirect3DPixelShader9 *pixel_shader[3] = {NULL, NULL, NULL}; - - /* This reference data was collected on a nVidia GeForce 7600GS driver version 84.19 DirectX version 9.0c on Windows XP */ - static const struct test_data_t { + /* This reference data was collected on a nVidia GeForce 7600GS driver + * version 84.19 DirectX version 9.0c on Windows XP. */ + static const struct test_data_t + { int vshader; int pshader; D3DFOGMODE vfog; D3DFOGMODE tfog; unsigned int color[11]; - } test_data[] = { + } + test_data[] = + { /* only pixel shader: */ {0, 1, D3DFOG_NONE, D3DFOG_LINEAR, {0x00ff0000, 0x00ff0000, 0x00df2000, 0x00bf4000, 0x009f6000, 0x007f8000, @@ -2010,10 +2017,43 @@ static void fog_with_shader_test(IDirect3DDevice9 *device) {0x00ff0000, 0x00ff0000, 0x00df2000, 0x00bf4000, 0x009f6000, 0x007f8000, 0x005fa000, 0x0040bf00, 0x0020df00, 0x0000ff00, 0x0000ff00}}, }; + static const D3DMATRIX identity = + {{{ + 1.0f, 0.0f, 0.0f, 0.0f, + 0.0f, 1.0f, 0.0f, 0.0f, + 0.0f, 0.0f, 1.0f, 0.0f, + 0.0f, 0.0f, 0.0f, 1.0f, + }}}; - /* NOTE: changing these values will not affect the tests with foggy vertex shader, as the values are hardcoded in the shader*/ - start.f=0.1f; - end.f=0.9f; + window = CreateWindowA("static", "d3d9_test", WS_OVERLAPPEDWINDOW | WS_VISIBLE, + 0, 0, 640, 480, NULL, NULL, NULL, NULL); + d3d = Direct3DCreate9(D3D_SDK_VERSION); + ok(!!d3d, "Failed to create a D3D object.\n"); + if (!(device = create_device(d3d, window, window, TRUE))) + { + skip("Failed to create a D3D device, skipping tests.\n"); + goto done; + } + + hr = IDirect3DDevice9_GetDeviceCaps(device, &caps); + ok(SUCCEEDED(hr), "Failed to get device caps, hr %#x.\n", hr); + if (caps.VertexShaderVersion < D3DVS_VERSION(2, 0) || caps.PixelShaderVersion < D3DPS_VERSION(2, 0)) + { + skip("No shader model 2 support, skipping tests.\n"); + IDirect3DDevice9_Release(device); + goto done; + } + + /* NOTE: Changing these values will not affect the tests with foggy vertex + * shader, as the values are hardcoded in the shader. */ + start.f = 0.1f; + end.f = 0.9f; + + /* Some of the tests seem to depend on the projection matrix explicitly + * being set to an identity matrix, even though that's the default. + * (AMD Radeon HD 6310, Windows 7) */ + hr = IDirect3DDevice9_SetTransform(device, D3DTS_PROJECTION, &identity); + ok(SUCCEEDED(hr), "Failed to set projection transform, hr %#x.\n", hr); hr = IDirect3DDevice9_CreateVertexShader(device, vertex_shader_code1, &vertex_shader[1]); ok(SUCCEEDED(hr), "CreateVertexShader failed (%08x)\n", hr); @@ -2068,7 +2108,7 @@ static void fog_with_shader_test(IDirect3DDevice9 *device) quad[2].z = 0.001f + (float)j / 10.02f; quad[3].z = 0.001f + (float)j / 10.02f; - hr = IDirect3DDevice9_Clear(device, 0, NULL, D3DCLEAR_TARGET, 0xffff00ff, 0.0, 0); + hr = IDirect3DDevice9_Clear(device, 0, NULL, D3DCLEAR_TARGET | D3DCLEAR_ZBUFFER, 0xffff00ff, 1.0f, 0); ok(hr == D3D_OK, "IDirect3DDevice9_Clear failed (%08x)\n", hr); hr = IDirect3DDevice9_BeginScene(device); @@ -2090,22 +2130,17 @@ static void fog_with_shader_test(IDirect3DDevice9 *device) } } - /* reset states */ - hr = IDirect3DDevice9_SetVertexShader(device, NULL); - ok(SUCCEEDED(hr), "SetVertexShader failed (%08x)\n", hr); - hr = IDirect3DDevice9_SetPixelShader(device, NULL); - ok(SUCCEEDED(hr), "SetPixelShader failed (%08x)\n", hr); - hr = IDirect3DDevice9_SetVertexDeclaration(device, NULL); - ok(SUCCEEDED(hr), "SetVertexDeclaration failed (%08x)\n", hr); - hr = IDirect3DDevice9_SetRenderState(device, D3DRS_FOGENABLE, FALSE); - ok(hr == D3D_OK, "Turning off fog calculations failed (%08x)\n", hr); - IDirect3DVertexShader9_Release(vertex_shader[1]); IDirect3DVertexShader9_Release(vertex_shader[2]); IDirect3DVertexShader9_Release(vertex_shader[3]); IDirect3DPixelShader9_Release(pixel_shader[1]); IDirect3DPixelShader9_Release(pixel_shader[2]); IDirect3DVertexDeclaration9_Release(vertex_declaration); + refcount = IDirect3DDevice9_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); +done: + IDirect3D9_Release(d3d); + DestroyWindow(window); } static void generate_bumpmap_textures(IDirect3DDevice9 *device) { @@ -16078,15 +16113,10 @@ START_TEST(visual) } else skip("No vs_2_0 support\n"); - if (caps.VertexShaderVersion >= D3DVS_VERSION(2, 0) && caps.PixelShaderVersion >= D3DPS_VERSION(2, 0)) - { - fog_with_shader_test(device_ptr); - } - else skip("No vs_2_0 and ps_2_0 support\n"); - cleanup_device(device_ptr); device_ptr = NULL; + fog_with_shader_test(); texbem_test(); texdepth_test(); texkill_test();

1 0