Move ugly .*.inc files into make/template/ where they can be more easily found

[user/henk/code/inspircd.git] / src / logger.cpp

diff --git a/src/logger.cpp b/src/logger.cpp
index 10b5591e71f7149527321ab6a91047f77f0235e9..19e07e78f64188b5b08e2724ced28d08f853409d 100644 (file)
--- a/src/logger.cpp
+++ b/src/logger.cpp
@@ -11,8 +11,6 @@
  * ---------------------------------------------------
  */
 
-/* $Core */
-
 #include "inspircd.h"
 
 #include "filelogger.h"
@@ -44,12 +42,28 @@
  *
  */
 
+LogManager::LogManager()
+{
+       noforkstream = NULL;
+       Logging = false;
+}
+
+LogManager::~LogManager()
+{
+       if (noforkstream)
+       {
+               ServerInstance->Logs = this;
+               delete noforkstream;
+               ServerInstance->Logs = NULL;
+       }
+}
+
 void LogManager::SetupNoFork()
 {
        if (!noforkstream)
        {
-               FileWriter* fw = new FileWriter(ServerInstance, stdout);
-               noforkstream = new FileLogStream(ServerInstance, ServerInstance->Config->forcedebug ? DEBUG : DEFAULT, fw);
+               FileWriter* fw = new FileWriter(stdout);
+               noforkstream = new FileLogStream(ServerInstance->Config->forcedebug ? DEBUG : DEFAULT, fw);
        }
        else
        {
@@ -70,18 +84,21 @@ void LogManager::OpenFileLogs()
        {
                return;
        }
-       ConfigReader* Conf = new ConfigReader(ServerInstance);
+       ConfigReader Conf;
        std::map<std::string, FileWriter*> logmap;
        std::map<std::string, FileWriter*>::iterator i;
-       for (int index = 0; index < Conf->Enumerate("log"); ++index)
+       for (int index = 0;; ++index)
        {
-               std::string method = Conf->ReadValue("log", "method", index);
+               ConfigTag* tag = ServerInstance->Config->ConfValue("log", index);
+               if (!tag)
+                       break;
+               std::string method = tag->getString("method");
                if (method != "file")
                {
                        continue;
                }
-               std::string type = Conf->ReadValue("log", "type", index);
-               std::string level = Conf->ReadValue("log", "level", index);
+               std::string type = tag->getString("type");
+               std::string level = tag->getString("level");
                int loglevel = DEFAULT;
                if (level == "debug" || ServerInstance->Config->forcedebug)
                {
@@ -105,18 +122,18 @@ void LogManager::OpenFileLogs()
                        loglevel = NONE;
                }
                FileWriter* fw;
-               std::string target = Conf->ReadValue("log", "target", index);
+               std::string target = Conf.ReadValue("log", "target", index);
                if ((i = logmap.find(target)) == logmap.end())
                {
                        FILE* f = fopen(target.c_str(), "a");
-                       fw = new FileWriter(ServerInstance, f);
+                       fw = new FileWriter(f);
                        logmap.insert(std::make_pair(target, fw));
                }
                else
                {
                        fw = i->second;
                }
-               FileLogStream* fls = new FileLogStream(ServerInstance, loglevel, fw);
+               FileLogStream* fls = new FileLogStream(loglevel, fw);
                AddLogTypes(type, fls, true);
        }
 }
@@ -330,8 +347,8 @@ void LogManager::Log(const std::string &type, int loglevel, const std::string &m
 }
 
 
-FileWriter::FileWriter(InspIRCd* Instance, FILE* logfile)
-: ServerInstance(Instance), log(logfile), writeops(0)
+FileWriter::FileWriter(FILE* logfile)
+: log(logfile), writeops(0)
 {
 }
 
@@ -341,13 +358,15 @@ void FileWriter::HandleEvent(EventType ev, int)
 
 void FileWriter::WriteLogLine(const std::string &line)
 {
-       if (log)
+       if (log == NULL)
+               return;
+// XXX: For now, just return. Don't throw an exception. It'd be nice to find out if this is happening, but I'm terrified of breaking so close to final release. -- w00t
+//             throw CoreException("FileWriter::WriteLogLine called with a closed logfile");
+
+       fprintf(log,"%s",line.c_str());
+       if (writeops++ % 20)
        {
-               fprintf(log,"%s",line.c_str());
-               if (writeops++ % 20)
-               {
-                       fflush(log);
-               }
+               fflush(log);
        }
 }
 
@@ -357,9 +376,11 @@ void FileWriter::Close()
        {
                fflush(log);
                fclose(log);
+               log = NULL;
        }
 }
 
 FileWriter::~FileWriter()
 {
+       this->Close();
 }