[PATCH 3/4] d3d9/tests: Test IDirect3D9::CheckDepthStencilMatch() with D3DFMT_D32.

[Jactry Zeng]

Signed-off-by: Jactry Zeng <jzeng(a)codeweavers.com>
---
 dlls/d3d9/tests/device.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/dlls/d3d9/tests/device.c b/dlls/d3d9/tests/device.c
index f5c89723cb..73ecae9804 100644
--- a/dlls/d3d9/tests/device.c
+++ b/dlls/d3d9/tests/device.c
@@ -11833,6 +11833,16 @@ static void test_check_device_format(void)
         }
     }
 
+    hr = IDirect3D9_CheckDepthStencilMatch(d3d, D3DADAPTER_DEFAULT, D3DDEVTYPE_HAL, D3DFMT_X8R8G8B8,
+            D3DFMT_A8R8G8B8, D3DFMT_D32);
+    ok(hr == D3DERR_NOTAVAILABLE || broken(hr == D3DERR_INVALIDCALL), /* Win10 */
+            "Got unexpected hr %#x.\n", hr);
+
+    hr = IDirect3D9_CheckDepthStencilMatch(d3d, D3DADAPTER_DEFAULT, D3DDEVTYPE_HAL, D3DFMT_X8R8G8B8,
+            D3DFMT_R5G6B5, D3DFMT_D32);
+    ok(hr == D3DERR_NOTAVAILABLE || broken(hr == D3DERR_INVALIDCALL), /* Win10 */
+            "Got unexpected hr %#x.\n", hr);
+
     IDirect3D9_Release(d3d);
 }
 
-- 
2.20.1

[Paul Gofman]

Please note that D3DFMT_D32 format is supported here on Win7 for d3d9 
reference device (please see attached patched on top of this one which 
checks that).

I am not sure if this is enough reason to keep it though. And I didn't 
test if the format is available for d3d10/11.

On 5/22/19 10:57, Jactry Zeng wrote:
> Signed-off-by: Jactry Zeng <jzeng(a)codeweavers.com>
> ---
>   dlls/d3d9/tests/device.c | 10 ++++++++++
>   1 file changed, 10 insertions(+)
>
> diff --git a/dlls/d3d9/tests/device.c b/dlls/d3d9/tests/device.c
> index f5c89723cb..73ecae9804 100644
> --- a/dlls/d3d9/tests/device.c
> +++ b/dlls/d3d9/tests/device.c
> @@ -11833,6 +11833,16 @@ static void test_check_device_format(void)
>           }
>       }
>
> +    hr = IDirect3D9_CheckDepthStencilMatch(d3d, D3DADAPTER_DEFAULT, D3DDEVTYPE_HAL, D3DFMT_X8R8G8B8,
> +            D3DFMT_A8R8G8B8, D3DFMT_D32);
> +    ok(hr == D3DERR_NOTAVAILABLE || broken(hr == D3DERR_INVALIDCALL), /* Win10 */
> +            "Got unexpected hr %#x.\n", hr);
> +
> +    hr = IDirect3D9_CheckDepthStencilMatch(d3d, D3DADAPTER_DEFAULT, D3DDEVTYPE_HAL, D3DFMT_X8R8G8B8,
> +            D3DFMT_R5G6B5, D3DFMT_D32);
> +    ok(hr == D3DERR_NOTAVAILABLE || broken(hr == D3DERR_INVALIDCALL), /* Win10 */
> +            "Got unexpected hr %#x.\n", hr);
> +
>       IDirect3D9_Release(d3d);
>   }

[Józef Kucia]

On Wed, May 22, 2019 at 1:36 PM Paul Gofman <gofmanp(a)gmail.com> wrote:
> Please note that D3DFMT_D32 format is supported here on Win7 for d3d9
> reference device (please see attached patched on top of this one which
> checks that).

D3DFMT_D32 is probably available only on reference devices.
> I am not sure if this is enough reason to keep it though. And I didn't
> test if the format is available for d3d10/11.

AFAIK, there is no DXGI_FORMAT for D32_UNORM.