Macro reportAlways also show message in log if asserts disabled in compile time.

1 files changed, 2 insertions, 18 deletions

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 ++;