Re: [PATCH 1/2] wined3d: make wined3d_device_copy_sub_resource_region return an error code

[Henri Verbeet]

On 28 October 2015 at 16:11, Stefan Dösinger  wrote:
> Am 2015-10-28 um 15:31 schrieb Riccardo Bortolato:
> > - if (dst_resource->type != WINED3D_RTYPE_TEXTURE)
> > - if (dst_resource->type == WINED3D_RTYPE_VOLUME_TEXTURE
> > -
> >        || src_resource->type == WINED3D_RTYPE_VOLUME_TEXTURE)
> > {
>
> I guess your intention is to avoid having a type == TEXTURE || type ==
> CUBE_TEXTURE kind of construct, but technically you'd also have to
> check for WINED3D_RTYPE_BUFFER. I'd prefer whitelisting known good
> resource types.

The issue mostly exists because d3d10 doesn't have cube textures as
such, but just generic 2D array textures. Ideally wined3d would be
fixed to match that first, but I suppose you could also just map
WINED3D_RTYPE_CUBE_TEXTURE to WINED3D_RTYPE_TEXTURE here.