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.