From fa7e1c8137cace0236ee5da2f59e42f51a71003b Mon Sep 17 00:00:00 2001 From: matto Date: Sat, 29 Aug 2015 11:55:34 -0400 Subject: [PATCH] Splitting up asserts If these asserts are to be useful for debugging, they should really be split up so that we can tell which one is failing. --- src/Textures.cpp | 33 ++++++++++++++++----------------- 1 file changed, 16 insertions(+), 17 deletions(-) diff --git a/src/Textures.cpp b/src/Textures.cpp index d3128ac1..50cc323a 100644 --- a/src/Textures.cpp +++ b/src/Textures.cpp @@ -1208,10 +1208,10 @@ void TextureCache::_updateBackground() Textures::iterator iter = m_textures.find(crc); if (iter != m_textures.end()) { CachedTexture & current = iter->second; - assert((current.width == gSP.bgImage.width) && - (current.height == gSP.bgImage.height) && - (current.format == gSP.bgImage.format) && - (current.size == gSP.bgImage.size)); + assert(current.width == gSP.bgImage.width); + assert(current.height == gSP.bgImage.height); + assert(current.format == gSP.bgImage.format); + assert(current.size == gSP.bgImage.size); activateTexture(0, ¤t); m_hits++; @@ -1353,19 +1353,18 @@ void TextureCache::update(u32 _t) Textures::iterator iter = m_textures.find(crc); if (iter != m_textures.end()) { CachedTexture & current = iter->second; - assert((current.width == sizes.width) && - (current.height == sizes.height) && - (current.clampWidth == sizes.clampWidth) && - (current.clampHeight == sizes.clampHeight) && - (current.maskS == gSP.textureTile[_t]->masks) && - (current.maskT == gSP.textureTile[_t]->maskt) && - (current.mirrorS == gSP.textureTile[_t]->mirrors) && - (current.mirrorT == gSP.textureTile[_t]->mirrort) && - (current.clampS == gSP.textureTile[_t]->clamps) && - (current.clampT == gSP.textureTile[_t]->clampt) && - (current.format == gSP.textureTile[_t]->format) && - (current.size == gSP.textureTile[_t]->size) - ); + assert(current.width == sizes.width); + assert(current.height == sizes.height); + assert(current.clampWidth == sizes.clampWidth); + assert(current.clampHeight == sizes.clampHeight); + assert(current.maskS == gSP.textureTile[_t]->masks); + assert(current.maskT == gSP.textureTile[_t]->maskt); + assert(current.mirrorS == gSP.textureTile[_t]->mirrors); + assert(current.mirrorT == gSP.textureTile[_t]->mirrort); + assert(current.clampS == gSP.textureTile[_t]->clamps); + assert(current.clampT == gSP.textureTile[_t]->clampt); + assert(current.format == gSP.textureTile[_t]->format); + assert(current.size == gSP.textureTile[_t]->size); activateTexture(_t, ¤t); m_hits++;