Code review feedback

This commit is contained in:
Chuck Walbourn 2022-05-18 15:18:54 -07:00
parent 3eed3ae913
commit 9a9fb5e39e
2 changed files with 21 additions and 32 deletions

View File

@ -1539,11 +1539,9 @@ namespace
//--------------------------------------------------------------------------------------
void SetDebugTextureInfo(
_In_z_ const wchar_t* fileName,
_In_ ID3D12Resource** texture) noexcept
_In_ ID3D12Resource* texture) noexcept
{
#if !defined(NO_D3D12_DEBUG_NAME) && ( defined(_DEBUG) || defined(PROFILE) )
if (texture && *texture)
{
const wchar_t* pstrName = wcsrchr(fileName, '\\');
if (!pstrName)
{
@ -1553,9 +1551,7 @@ namespace
{
pstrName++;
}
(*texture)->SetName(pstrName);
}
texture->SetName(pstrName);
#else
UNREFERENCED_PARAMETER(fileName);
UNREFERENCED_PARAMETER(texture);
@ -1642,11 +1638,8 @@ HRESULT DirectX::LoadDDSTextureFromMemoryEx(
resFlags, loadFlags,
texture, subresources, isCubeMap);
if (SUCCEEDED(hr))
{
if (texture && *texture)
{
SetDebugObjectName(*texture, L"DDSTextureLoader");
}
if (alphaMode)
*alphaMode = GetAlphaMode(header);
@ -1734,7 +1727,7 @@ HRESULT DirectX::LoadDDSTextureFromFileEx(
if (SUCCEEDED(hr))
{
SetDebugTextureInfo(fileName, texture);
SetDebugTextureInfo(fileName, *texture);
if (alphaMode)
*alphaMode = GetAlphaMode(header);

View File

@ -640,11 +640,9 @@ namespace
//--------------------------------------------------------------------------------------
void SetDebugTextureInfo(
_In_z_ const wchar_t* fileName,
_In_ ID3D12Resource** texture) noexcept
_In_ ID3D12Resource* texture) noexcept
{
#if !defined(NO_D3D12_DEBUG_NAME) && ( defined(_DEBUG) || defined(PROFILE) )
if (texture && *texture)
{
const wchar_t* pstrName = wcsrchr(fileName, '\\');
if (!pstrName)
{
@ -654,9 +652,7 @@ namespace
{
pstrName++;
}
(*texture)->SetName(pstrName);
}
texture->SetName(pstrName);
#else
UNREFERENCED_PARAMETER(fileName);
UNREFERENCED_PARAMETER(texture);
@ -822,7 +818,7 @@ HRESULT DirectX::LoadWICTextureFromFileEx(
if (SUCCEEDED(hr))
{
SetDebugTextureInfo(fileName, texture);
SetDebugTextureInfo(fileName, *texture);
}
return hr;