From d1b132731dcb81aa4283564a5f9ff40a4c6b2047 Mon Sep 17 00:00:00 2001 From: Stenzek Date: Tue, 24 Nov 2015 21:54:19 +1000 Subject: [PATCH] D3D: Remove redundant cast, move pitch calculation to caller --- Source/Core/VideoBackends/D3D/D3DTexture.cpp | 5 ++--- Source/Core/VideoBackends/D3D/D3DTexture.h | 2 +- Source/Core/VideoBackends/D3D/TextureCache.cpp | 3 ++- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/Source/Core/VideoBackends/D3D/D3DTexture.cpp b/Source/Core/VideoBackends/D3D/D3DTexture.cpp index eb5d0d594a..b5a65c9a39 100644 --- a/Source/Core/VideoBackends/D3D/D3DTexture.cpp +++ b/Source/Core/VideoBackends/D3D/D3DTexture.cpp @@ -15,9 +15,8 @@ namespace DX11 namespace D3D { -void ReplaceRGBATexture2D(ID3D11Texture2D* pTexture, const u8* buffer, unsigned int width, unsigned int height, unsigned int expanded_width, unsigned int level, D3D11_USAGE usage) +void ReplaceRGBATexture2D(ID3D11Texture2D* pTexture, const u8* buffer, unsigned int width, unsigned int height, unsigned int src_pitch, unsigned int level, D3D11_USAGE usage) { - unsigned int src_pitch = 4 * expanded_width; if (usage == D3D11_USAGE_DYNAMIC || usage == D3D11_USAGE_STAGING) { D3D11_MAPPED_SUBRESOURCE map; @@ -32,7 +31,7 @@ void ReplaceRGBATexture2D(ID3D11Texture2D* pTexture, const u8* buffer, unsigned // pitch to what the driver returns, so copy whichever is smaller. unsigned int copy_size = std::min(src_pitch, map.RowPitch); for (unsigned int y = 0; y < height; ++y) - memcpy((u8*)map.pData + y * map.RowPitch, (u8*)buffer + y * src_pitch, copy_size); + memcpy((u8*)map.pData + y * map.RowPitch, buffer + y * src_pitch, copy_size); } D3D::context->Unmap(pTexture, level); } diff --git a/Source/Core/VideoBackends/D3D/D3DTexture.h b/Source/Core/VideoBackends/D3D/D3DTexture.h index 7118d5d979..86d7c188a4 100644 --- a/Source/Core/VideoBackends/D3D/D3DTexture.h +++ b/Source/Core/VideoBackends/D3D/D3DTexture.h @@ -11,7 +11,7 @@ namespace DX11 namespace D3D { - void ReplaceRGBATexture2D(ID3D11Texture2D* pTexture, const u8* buffer, unsigned int width, unsigned int height, unsigned int expanded_width, unsigned int level, D3D11_USAGE usage); + void ReplaceRGBATexture2D(ID3D11Texture2D* pTexture, const u8* buffer, unsigned int width, unsigned int height, unsigned int src_pitch, unsigned int level, D3D11_USAGE usage); } class D3DTexture2D diff --git a/Source/Core/VideoBackends/D3D/TextureCache.cpp b/Source/Core/VideoBackends/D3D/TextureCache.cpp index 1d5141ce0e..0de91068c7 100644 --- a/Source/Core/VideoBackends/D3D/TextureCache.cpp +++ b/Source/Core/VideoBackends/D3D/TextureCache.cpp @@ -142,7 +142,8 @@ void TextureCache::TCacheEntry::CopyRectangleFromTexture( void TextureCache::TCacheEntry::Load(unsigned int width, unsigned int height, unsigned int expanded_width, unsigned int level) { - D3D::ReplaceRGBATexture2D(texture->GetTex(), TextureCache::temp, width, height, expanded_width, level, usage); + unsigned int src_pitch = 4 * expanded_width; + D3D::ReplaceRGBATexture2D(texture->GetTex(), TextureCache::temp, width, height, src_pitch, level, usage); } TextureCacheBase::TCacheEntryBase* TextureCache::CreateTexture(const TCacheEntryConfig& config)