From: Florian Forster <supertux@octo.it>
Date: Thu, 11 Feb 2010 16:07:56 +0000 (+0000)
Subject: Bug 574: Check whether the Console instance still exists before accessing it.
X-Git-Url: https://git.verplant.org/?a=commitdiff_plain;h=66f0d5ddf530aa29518056bde144c76ab700f157;p=supertux.git

Bug 574: Check whether the Console instance still exists before accessing it.

If the global Console instance doesn't exist anymore (or not yet), "std::cerr"
will be used instead.

Resolves: #574.

SVN-Revision: 6322
---

diff --git a/src/util/log.cpp b/src/util/log.cpp
index 036ada58c..6ce6f73b4 100644
--- a/src/util/log.cpp
+++ b/src/util/log.cpp
@@ -1,5 +1,6 @@
 //  SuperTux Debug Helper Functions
 //  Copyright (C) 2006 Christoph Sommer <christoph.sommer@2006.expires.deltadevelopment.de>
+//  Copyright (C) 2010 Florian Forster <supertux at octo.it>
 //
 //  This program is free software: you can redistribute it and/or modify
 //  it under the terms of the GNU General Public License as published by
@@ -16,33 +17,43 @@
 
 #include "util/log.hpp"
 
+#include <iostream>
+
 #include "math/rectf.hpp"
 #include "supertux/console.hpp"
 
+static std::ostream& get_logging_instance (void)
+{
+  if (Console::instance != NULL)
+    return (Console::output);
+  else
+    return (std::cerr);
+}
+
+static std::ostream& log_generic_f (const char *prefix, const char* file, int line)
+{
+  get_logging_instance () << prefix << " " << file << ":" << line << " ";
+  return (get_logging_instance ());
+}
+
 std::ostream& log_debug_f(const char* file, int line) 
 {
-  Console::output << "[DEBUG] " << file << ":" << line << " ";
-  return Console::output;
+  return (log_generic_f ("[DEBUG]", file, line));
 }
 
 std::ostream& log_info_f(const char* file, int line) 
 {
-  Console::output << "[INFO] " << file << ":" << line << " ";
-  return Console::output;
+  return (log_generic_f ("[INFO]", file, line));
 }
 
 std::ostream& log_warning_f(const char* file, int line) 
 {
-  Console::instance->open();
-  Console::output << "[WARNING] " << file << ":" << line << " ";
-  return Console::output;
+  return (log_generic_f ("[WARNING]", file, line));
 }
 
 std::ostream& log_fatal_f(const char* file, int line) 
 {
-  Console::instance->open();
-  Console::output << "[FATAL] " << file << ":" << line << " ";
-  return Console::output;
+  return (log_generic_f ("[FATAL]", file, line));
 }
 
 std::ostream& operator<<(std::ostream& out, const Vector& vector)