Re: [PATCH 0/7] MR5666: d3dx9: Introduce d3dx_resource structures for use in code sharing.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/d3dx9_private.h:
>  D3DX_RESOURCE_TYPE_IMAGE = 1,
>
> };
>
> +struct d3dx_sub_resource

Another naming comment... I'm not crazy about the `d3dx_sub_resource` name: it feels a bit awkward when it's used "standalone", not immediately as part of a `d3dx_resource` (I think you setup a throw-away d3dx_sub_resource in a later patch).

I don't have great suggestions for this one. Maybe `d3dx_image` or `d3dx_surface`, but neither sounds super appealing to me.

Another option would be to enforce every `d3dx_sub_resource` to be tied to a `d3dx_resource`. It doesn't seem to be a better plan at a glance but I haven't given it much thought at this point.