From: ThorstenB <brehmt@gmail.com>
Date: Sun, 3 Jul 2011 11:04:00 +0000 (+0200)
Subject: Alert message formatting.
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=d4f5a35e87770e597ef52c3caa89dedfe738aa8b;p=simgear.git

Alert message formatting.
Avoid log output cluttering due to "missing animation objects".
---

diff --git a/simgear/scene/model/animation.cxx b/simgear/scene/model/animation.cxx
index 35113342..fa9b44e1 100644
--- a/simgear/scene/model/animation.cxx
+++ b/simgear/scene/model/animation.cxx
@@ -386,14 +386,24 @@ SGAnimation::SGAnimation(const SGPropertyNode* configNode,
 
 SGAnimation::~SGAnimation()
 {
-  if (_found)
-    return;
-
-  SG_LOG(SG_IO, SG_ALERT, "Could not find at least one of the following"
-         " objects for animation:\n");
-  std::list<std::string>::const_iterator i;
-  for (i = _objectNames.begin(); i != _objectNames.end(); ++i)
-    SG_LOG(SG_IO, SG_ALERT, *i << "\n");
+  if (!_found)
+  {
+      std::list<std::string>::const_iterator i;
+      string info;
+      for (i = _objectNames.begin(); i != _objectNames.end(); ++i)
+      {
+          if (!info.empty())
+              info.append(", ");
+          info.append("'");
+          info.append(*i);
+          info.append("'");
+      }
+      if (!info.empty())
+      {
+          SG_LOG(SG_IO, SG_ALERT, "Could not find at least one of the following"
+                  " objects for animation: " << info);
+      }
+  }
 }
 
 bool