d3d8: Return D3D_OK from UnlockRect() for not locked textures.
Signed-off-by: Józef Kucia <jkucia@codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
parent
52452b271e
commit
8f8c2a8ade
|
@ -271,11 +271,16 @@ static HRESULT WINAPI d3d8_surface_UnlockRect(IDirect3DSurface8 *iface)
|
|||
hr = wined3d_resource_unmap(wined3d_texture_get_resource(surface->wined3d_texture), surface->sub_resource_idx);
|
||||
wined3d_mutex_unlock();
|
||||
|
||||
switch(hr)
|
||||
if (hr == WINEDDERR_NOTLOCKED)
|
||||
{
|
||||
case WINEDDERR_NOTLOCKED: return D3DERR_INVALIDCALL;
|
||||
default: return hr;
|
||||
D3DRESOURCETYPE type;
|
||||
if (surface->texture)
|
||||
type = IDirect3DBaseTexture8_GetType(&surface->texture->IDirect3DBaseTexture8_iface);
|
||||
else
|
||||
type = D3DRTYPE_SURFACE;
|
||||
hr = type == D3DRTYPE_TEXTURE ? D3D_OK : D3DERR_INVALIDCALL;
|
||||
}
|
||||
return hr;
|
||||
}
|
||||
|
||||
static const IDirect3DSurface8Vtbl d3d8_surface_vtbl =
|
||||
|
|
|
@ -4883,7 +4883,6 @@ static void test_lockrect_invalid(void)
|
|||
ok(SUCCEEDED(hr), "Failed to unlock surface, hr %#x, type %s.\n", hr, resources[r].name);
|
||||
expected_hr = resources[r].type == D3DRTYPE_TEXTURE ? D3D_OK : D3DERR_INVALIDCALL;
|
||||
hr = IDirect3DSurface8_UnlockRect(surface);
|
||||
todo_wine_if(resources[r].type == D3DRTYPE_TEXTURE)
|
||||
ok(hr == expected_hr, "Got hr %#x, expected %#x, type %s.\n", hr, expected_hr, resources[r].name);
|
||||
|
||||
for (i = 0; i < (sizeof(valid) / sizeof(*valid)); ++i)
|
||||
|
@ -5025,7 +5024,7 @@ static void test_lockrect_invalid(void)
|
|||
hr = IDirect3DTexture8_UnlockRect(texture, 0);
|
||||
ok(SUCCEEDED(hr), "Failed to unlock texture, hr %#x, type %s.\n", hr, resources[r].name);
|
||||
hr = IDirect3DTexture8_UnlockRect(texture, 0);
|
||||
todo_wine ok(hr == D3D_OK, "Got unexpected hr %#x, type %s.\n", hr, resources[r].name);
|
||||
ok(hr == D3D_OK, "Got unexpected hr %#x, type %s.\n", hr, resources[r].name);
|
||||
|
||||
hr = IDirect3DTexture8_LockRect(texture, 0, &locked_rect, &valid[0], 0);
|
||||
ok(hr == D3D_OK, "Got unexpected hr %#x for rect %s, type %s.\n",
|
||||
|
|
Loading…
Reference in New Issue