Re: [PATCH v5 0/4] MR627: vkd3d: Implement ID3D12Resource2.

[Henri Verbeet (＠hverbeet)]

> @@ -1723,9 +1723,9 @@ static HRESULT STDMETHODCALLTYPE d3d12_resource_GetHeapProperties(ID3D12Resource
>  static HRESULT STDMETHODCALLTYPE d3d12_resource_GetProtectedResourceSession(ID3D12Resource1 *iface,
>          REFIID iid, void **session)
>  {
> -    FIXME("iface %p, iid %s, session %p stub!\n", iface, debugstr_guid(iid), session);
> +    TRACE("iface %p, iid %s, session %p.\n", iface, debugstr_guid(iid), session);

Is changing the FIXME to a TRACE really warranted? Sure, it'll return the right result now, but that's only because we don't implement creating protected resources.
> +static void d3d12_resource_desc_promote(D3D12_RESOURCE_DESC1 *desc1, const D3D12_RESOURCE_DESC *desc)
> +{
> +    memcpy(desc1, desc, sizeof(*desc));
> +    desc1->SamplerFeedbackMipRegion.Width = 0;
> +    desc1->SamplerFeedbackMipRegion.Height = 0;
> +    desc1->SamplerFeedbackMipRegion.Depth = 0;
> +}

Could we call that "d3d12_resource_desc1_from_desc()", please?