From 7d762d70604d250cb51282161d390cba1ea664c7 Mon Sep 17 00:00:00 2001 From: Andrei Karas Date: Thu, 12 May 2016 15:08:15 +0300 Subject: Macro reportAlways also show message in log if asserts disabled in compile time. --- src/resources/sprite/spritedef.cpp | 20 ++------------------ 1 file changed, 2 insertions(+), 18 deletions(-) (limited to 'src/resources/sprite') diff --git a/src/resources/sprite/spritedef.cpp b/src/resources/sprite/spritedef.cpp index 312597789..ae7fb2580 100644 --- a/src/resources/sprite/spritedef.cpp +++ b/src/resources/sprite/spritedef.cpp @@ -100,7 +100,6 @@ SpriteDef *SpriteDef::load(const std::string &animationFile, if (!rootNode || !xmlNameEqual(rootNode, "sprite")) { - logger->log("Error, failed to parse sprite %s", animationFile.c_str()); reportAlways("Error, failed to parse sprite %s", animationFile.c_str()); const std::string errorFile = paths.getStringValue("sprites").append( @@ -245,7 +244,6 @@ void SpriteDef::loadImageSet(const XmlNodePtr node, if (!imageSet) { - logger->log1("Couldn't load imageset!"); reportAlways("Couldn't load imageset!"); return; } @@ -268,8 +266,6 @@ void SpriteDef::loadAction(const XmlNodePtr node, const ImageSetIterator si = mImageSets.find(imageSetName); if (si == mImageSets.end()) { - logger->log("Warning: imageset \"%s\" not defined in %s", - imageSetName.c_str(), getIdPath().c_str()); reportAlways("Warning: imageset \"%s\" not defined in %s", imageSetName.c_str(), getIdPath().c_str()); return; @@ -278,8 +274,6 @@ void SpriteDef::loadAction(const XmlNodePtr node, if (actionName == SpriteAction::INVALID) { - logger->log("Warning: Unknown action \"%s\" defined in %s", - actionName.c_str(), getIdPath().c_str()); reportAlways("Warning: Unknown action \"%s\" defined in %s", actionName.c_str(), getIdPath().c_str()); return; @@ -320,10 +314,9 @@ void SpriteDef::loadAnimation(const XmlNodePtr animationNode, if (directionType == SpriteDirection::INVALID) { - logger->log("Warning: Unknown direction \"%s\" used in %s", - directionName.c_str(), getIdPath().c_str()); reportAlways("Warning: Unknown direction \"%s\" used in %s", - directionName.c_str(), getIdPath().c_str()); + directionName.c_str(), + getIdPath().c_str()); return; } @@ -454,8 +447,6 @@ void SpriteDef::includeSprite(const XmlNodePtr includeNode, const int variant) if (mProcessedFiles.find(filename) != mProcessedFiles.end()) { - logger->log("Error, Tried to include %s which already is included.", - filename.c_str()); reportAlways("Error, Tried to include %s which already is included.", filename.c_str()); return; @@ -467,7 +458,6 @@ void SpriteDef::includeSprite(const XmlNodePtr includeNode, const int variant) if (!rootNode || !xmlNameEqual(rootNode, "sprite")) { - logger->log("Error, no sprite root node in %s", filename.c_str()); reportAlways("Error, no sprite root node in %s", filename.c_str()); return; } @@ -553,7 +543,6 @@ bool SpriteDef::addSequence(const int start, if (start < 0 || end < 0) { - logger->log1("No valid value for 'start' or 'end'"); reportAlways("No valid value for 'start' or 'end'"); return true; } @@ -569,9 +558,6 @@ bool SpriteDef::addSequence(const int start, if (!img) { - logger->log("%s: No image at index %d", - mSource.c_str(), - pos + variant_offset); reportAlways("%s: No image at index %d", mSource.c_str(), pos + variant_offset); @@ -597,8 +583,6 @@ bool SpriteDef::addSequence(const int start, if (!img) { - logger->log("No image at index %d", - pos + variant_offset); reportAlways("No image at index %d", pos + variant_offset); pos ++; -- cgit v1.2.3-60-g2f50