[PATCH v3 1/2] d3d8: Report d3d8 adapter ordinal in d3d8_device_GetCreationParameters().

[Zhiyi Zhang]

Signed-off-by: Zhiyi Zhang 
---
v3: Supersede 181455~181456. Remove behaviour flags tests.

dlls/d3d8/device.c       |  1 +
 dlls/d3d8/tests/device.c | 48 ++++++++++++++++++++++++++++++++++++++++
 2 files changed, 49 insertions(+)

diff --git a/dlls/d3d8/device.c b/dlls/d3d8/device.c
index 400e1e292a..23e93df061 100644
--- a/dlls/d3d8/device.c
+++ b/dlls/d3d8/device.c
@@ -730,6 +730,7 @@ static HRESULT WINAPI d3d8_device_GetCreationParameters(IDirect3DDevice8 *iface,
             (struct wined3d_device_creation_parameters *)parameters);
     wined3d_mutex_unlock();

+    parameters->AdapterOrdinal = device->adapter_ordinal;
     return D3D_OK;
 }

diff --git a/dlls/d3d8/tests/device.c b/dlls/d3d8/tests/device.c
index 2e182e6192..a2c02b2c6a 100644
--- a/dlls/d3d8/tests/device.c
+++ b/dlls/d3d8/tests/device.c
@@ -9869,6 +9869,53 @@ static void test_multi_adapter(void)
     IDirect3D8_Release(d3d);
 }

+static void test_creation_parameters(void)
+{
+    unsigned int adapter_idx, adapter_count;
+    D3DDEVICE_CREATION_PARAMETERS params;
+    struct device_desc device_desc;
+    IDirect3DDevice8 *device;
+    IDirect3D8 *d3d;
+    HWND window;
+    HRESULT hr;
+
+    window = create_window();
+    ok(!!window, "Failed to create a window.\n");
+    d3d = Direct3DCreate8(D3D_SDK_VERSION);
+    ok(!!d3d, "Failed to create a D3D object.\n");
+
+    device_desc.device_window = window;
+    device_desc.width = 640;
+    device_desc.height = 480;
+    device_desc.flags = 0;
+
+    adapter_count = IDirect3D8_GetAdapterCount(d3d);
+    for (adapter_idx = 0; adapter_idx < adapter_count; ++adapter_idx)
+    {
+        device_desc.adapter_ordinal = adapter_idx;
+        if (!(device = create_device(d3d, window, &device_desc)))
+        {
+            skip("Adapter %u: Failed to create a D3D device.\n", adapter_idx);
+            break;
+        }
+
+        memset(&params, 0, sizeof(params));
+        hr = IDirect3DDevice8_GetCreationParameters(device, &params);
+        ok(hr == D3D_OK, "Adapter %u: GetCreationParameters failed, hr %#x.\n", adapter_idx, hr);
+        ok(params.AdapterOrdinal == adapter_idx, "Adapter %u: Got unexpected adapter ordinal %u.\n",
+                adapter_idx, params.AdapterOrdinal);
+        ok(params.DeviceType == D3DDEVTYPE_HAL, "Adapter %u: Expect device type %#x, got %#x.\n",
+                adapter_idx, D3DDEVTYPE_HAL, params.DeviceType);
+        ok(params.hFocusWindow == window, "Adapter %u: Expect focus window %p, got %p.\n",
+                adapter_idx, window, params.hFocusWindow);
+
+        IDirect3DDevice8_Release(device);
+    }
+
+    IDirect3D8_Release(d3d);
+    DestroyWindow(window);
+}
+
 START_TEST(device)
 {
     HMODULE d3d8_handle = GetModuleHandleA("d3d8.dll");
@@ -9986,6 +10033,7 @@ START_TEST(device)
     test_draw_primitive();
     test_get_display_mode();
     test_multi_adapter();
+    test_creation_parameters();

UnregisterClassA("d3d8_test_wc", GetModuleHandleA(NULL));
 }

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=67766

Your paranoid android.

=== w1064v1809_ar (32 bit report) ===

d3d8:
device.c:1002: Test failed: The cursor handle is 00010019

[Henri Verbeet]

Signed-off-by: Henri Verbeet

[Henri Verbeet]

Signed-off-by: Henri Verbeet