Re: [PATCH v6 0/4] MR5161: d3drm: Implement Update and related functions

[Zebediah Figura (＠zfigura)]

Zebediah Figura (@zfigura) commented about dlls/d3drm/tests/d3drm.c:
> +    struct update_surface_context ctx;
> +    D3DCOLOR ret_color = 0;
> +    DWORD clip_list_size;
> +    RGNDATA *clip_list;
> +    HWND window;
> +    HRESULT hr;
> +
> +    window = create_window();
> +
> +    /* Set up device */
> +    hr = DirectDrawCreateClipper(0, &clipper, NULL);
> +    ok(hr == DD_OK, "Cannot get IDirectDrawClipper interface, hr %#lx.\n", hr);
> +    hr = IDirectDrawClipper_SetHWnd(clipper, 0, window);
> +    ok(hr == DD_OK, "Cannot set HWnd to Clipper, hr %#lx.\n", hr);
> +    hr = Direct3DRMCreate(&d3drm1);
> +    ok(SUCCEEDED(hr), "Cannot create IDirect3DRM instance, hr %#lx.\n", hr);
> Not a big deal, but in the future I'd like to avoid this, and just always check the actual hr value.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5161#note_63572