ConsoleLogger bug

While tinkering with the ConsoleLogger class I stumbled on a bug in it:
When using more then one ConsoleLogger, the logger writes double lines (or as many ConsoleLoggers as you have active) into the log file.

This error is the result of the console callback function getting registered multiple times ( Con::addConsumer( ConsoleLogger::logCallback ); ), instead of just once for all ConsoleLoggers.

A fix is to make the mInitialized member of the ConsoleLogger a static field:

consoleLogger.h:

private:
      bool mLogging;         ///< True if it is currently consuming and logging
      FileStream mStream;    ///< File stream this object writes to
      bool mInitalized;      ///< This is for use with the default constructor
      bool mAppend;          ///< If false, it will clear the file before logging to it.
      StringTableEntry mFilename;      ///< The file name to log to.

to

private:
      bool mLogging;             ///< True if it is currently consuming and logging
      FileStream mStream;        ///< File stream this object writes to
      static bool mInitalized;   ///< This is for use with the default constructor
      bool mAppend;              ///< If false, it will clear the file before logging to it.
      StringTableEntry mFilename;      ///< The file name to log to.

To initialize the static member of the object we need to add 'bool ConsoleLogger::mInitalized = false;' to consoleLogger.cc:
consoleLogger.cc:

#include "console/consoleLogger.h"
#include "console/consoleTypes.h"

Vector<ConsoleLogger *> ConsoleLogger::mActiveLoggers;
bool ConsoleLogger::mInitalized = false;

IMPLEMENT_CONOBJECT( ConsoleLogger );

And then altering the init() method of the ConsoleLogger to:
consoleLogger.cc:

bool ConsoleLogger::init() {
   if( mInitalized )
      return true;

   Con::addConsumer( ConsoleLogger::logCallback );
   mInitalized = true;

   return true;
}

For tidyness I moved the 'mFilename == NULL' check from the init() method to the attach() method :
consoleLogger.cc:

bool ConsoleLogger::attach() {

   // Check to see if this is intalized before using it
   if( !mInitalized ) {
      if( !init() ) {

to:

bool ConsoleLogger::attach() {

   if( mFilename == NULL )
   {
      Con::errorf( "ConsoleLogger failed to attach: no filename supplied." );
      return false;
   }

   // Check to see if this is intalized before using it
   if( !mInitalized ) {
      if( !init() ) {

There are also some other improvements which can be implemented, like removing the registered callback function from the Con::removeConsumer() when there are no ConsoleLoggers active anymore, but I'll keep it at a bugfix for now.