Use log filenames w/ timestamps instead of adding numbers to their ends

apps/openmw-mp/main.cpp

@@ -9,6 +9,7 @@
 #include <components/openmw-mp/NetworkMessages.hpp>
 #include <apps/openmw-mp/Script/Script.hpp>
 #include <iostream>
+#include <ctime>
 #include <components/files/configurationmanager.hpp>
 #include <components/settings/settings.hpp>
 #include <boost/iostreams/concepts.hpp>
@@ -171,10 +172,15 @@ int main(int argc, char *argv[])
 
     boost::filesystem::ofstream logfile;
 
-    // Redirect cout and cerr to openmw.log
-    cout << Log::renameOldLog(cfgMgr.getLogPath(), "server", "log") << endl;
+    // Get timestamp to add at the end of the log's filename
+    time_t rawtime = time(0);
+    struct tm *timeinfo = localtime(&rawtime);
+    char buffer[25];
+    strftime(buffer, 25, "%Y-%m-%d-%I_%M_%S", timeinfo);
+    std::string timestamp(buffer);
 
-    logfile.open (boost::filesystem::path(cfgMgr.getLogPath() / "/server.log"));
+    // Redirect cout and cerr to tes3mp server log
+    logfile.open (boost::filesystem::path(cfgMgr.getLogPath() / "/tes3mp-server-" += timestamp += ".log"));
 
     coutsb.open (Tee(logfile, oldcout));
     cerrsb.open (Tee(logfile, oldcerr));

apps/openmw/main.cpp

@@ -1,5 +1,6 @@
 #include <iostream>
 #include <cstdio>
+#include <ctime>
 
 #include <components/version/version.hpp>
 #include <components/files/configurationmanager.hpp>
@@ -334,9 +335,15 @@ int main(int argc, char**argv)
         std::cout.rdbuf (&sb);
         std::cerr.rdbuf (&sb);
 #else
-        // Redirect cout and cerr to openmw.log
-        Log::renameOldLog(cfgMgr.getLogPath(), "openmw", "log");
-        logfile.open (boost::filesystem::path(cfgMgr.getLogPath() / "/openmw.log"));
+        // Get timestamp to add at the end of the log's filename
+        time_t rawtime = time(0);
+        struct tm *timeinfo = localtime(&rawtime);
+        char buffer[25];
+        strftime(buffer, 25, "%Y-%m-%d-%I_%M_%S", timeinfo);
+        std::string timestamp(buffer);
+
+        // Redirect cout and cerr to tes3mp client log
+        logfile.open (boost::filesystem::path(cfgMgr.getLogPath() / "/tes3mp-client-" += timestamp += ".log"));
 
         coutsb.open (Tee(logfile, oldcout));
         cerrsb.open (Tee(logfile, oldcerr));

components/openmw-mp/Log.cpp

@@ -105,20 +105,3 @@ void Log::print(int level, bool hasPrefix, const char *file, int line, const cha
     va_end(args);
     cout << buf.data() << flush;
 }
-
-string Log::renameOldLog(boost::filesystem::path path, string name, string extension)
-{
-    namespace fs = boost::filesystem;
-
-    std::string fullname = name + "." + extension;
-
-    if (fs::exists(path / fullname))
-    {
-        std::string newname;
-        for (int i = 1; fs::exists(path / (newname = name + boost::lexical_cast<string>(i++) + "." + extension)););
-
-        fs::rename(path / fullname, path / newname);
-        return newname;
-    }
-    return fullname;
-}

components/openmw-mp/Log.hpp

@@ -46,8 +46,6 @@ public:
     static const Log &Get();
     static void SetLevel(int level);
     void print(int level, bool hasPrefix, const char *file, int line, const char *message, ...) const;
-
-    static std::string renameOldLog(boost::filesystem::path path, std::string name, std::string extension);
 private:
     Log(int logLevel);
     /// Not implemented