diff --git a/.gitignore b/.gitignore index 4009b5a..14ab1c8 100644 --- a/.gitignore +++ b/.gitignore @@ -15,6 +15,5 @@ RemoteCommand *.user *.VC.db .vs -.vscode *.ambe GitVersion.h diff --git a/Conf.cpp b/Conf.cpp index a056182..7491717 100644 --- a/Conf.cpp +++ b/Conf.cpp @@ -82,7 +82,6 @@ m_logDisplayLevel(0U), m_logFileLevel(0U), m_logFilePath(), m_logFileRoot(), -m_logTimestampLogs(1U), m_cwIdEnabled(false), m_cwIdTime(10U), m_cwIdCallsign(), @@ -454,8 +453,6 @@ bool CConf::read() m_logFileLevel = (unsigned int)::atoi(value); else if (::strcmp(key, "DisplayLevel") == 0) m_logDisplayLevel = (unsigned int)::atoi(value); - else if (::strcmp(key, "TimestampLogs") == 0) - m_logTimestampLogs = (unsigned int)::atoi(value); } else if (section == SECTION_CWID) { if (::strcmp(key, "Enable") == 0) m_cwIdEnabled = ::atoi(value) == 1; @@ -1077,11 +1074,6 @@ std::string CConf::getLogFileRoot() const return m_logFileRoot; } -unsigned int CConf::getLogTimestampLogs() const -{ - return m_logTimestampLogs; -} - bool CConf::getCWIdEnabled() const { return m_cwIdEnabled; diff --git a/Conf.h b/Conf.h index c7b611a..335d461 100644 --- a/Conf.h +++ b/Conf.h @@ -54,7 +54,6 @@ public: unsigned int getLogFileLevel() const; std::string getLogFilePath() const; std::string getLogFileRoot() const; - unsigned int getLogTimestampLogs() const; // The CW ID section bool getCWIdEnabled() const; @@ -338,7 +337,6 @@ private: unsigned int m_logFileLevel; std::string m_logFilePath; std::string m_logFileRoot; - unsigned int m_logTimestampLogs; bool m_cwIdEnabled; unsigned int m_cwIdTime; diff --git a/Log.cpp b/Log.cpp index c0a0162..1d5ad29 100644 --- a/Log.cpp +++ b/Log.cpp @@ -41,8 +41,6 @@ static bool m_daemon = false; static unsigned int m_displayLevel = 2U; -static unsigned int m_timestampLogs = 1U; - static struct tm m_tm; static char LEVELS[] = " DMIWEF"; @@ -68,16 +66,10 @@ static bool LogOpen() } char filename[200U]; - char timestamp[37U] = ""; - - if (m_timestampLogs) { - ::sprintf(timestamp, "-%04d-%02d-%02d", tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday); - } - #if defined(_WIN32) || defined(_WIN64) - ::sprintf(filename, "%s\\%s%s.log", m_filePath.c_str(), m_fileRoot.c_str(), timestamp); + ::sprintf(filename, "%s\\%s-%04d-%02d-%02d.log", m_filePath.c_str(), m_fileRoot.c_str(), tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday); #else - ::sprintf(filename, "%s/%s%s.log", m_filePath.c_str(), m_fileRoot.c_str(), timestamp); + ::sprintf(filename, "%s/%s-%04d-%02d-%02d.log", m_filePath.c_str(), m_fileRoot.c_str(), tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday); #endif if ((m_fpLog = ::fopen(filename, "a+t")) != NULL) { @@ -94,13 +86,12 @@ static bool LogOpen() return status; } -bool LogInitialise(bool daemon, const std::string& filePath, const std::string& fileRoot, unsigned int fileLevel, unsigned int displayLevel, unsigned int timestampLogs) +bool LogInitialise(bool daemon, const std::string& filePath, const std::string& fileRoot, unsigned int fileLevel, unsigned int displayLevel) { m_filePath = filePath; m_fileRoot = fileRoot; m_fileLevel = fileLevel; m_displayLevel = displayLevel; - m_timestampLogs = timestampLogs; m_daemon = daemon; if (m_daemon) diff --git a/Log.h b/Log.h index f287f35..0d00653 100644 --- a/Log.h +++ b/Log.h @@ -30,7 +30,7 @@ extern void Log(unsigned int level, const char* fmt, ...); -extern bool LogInitialise(bool daemon, const std::string& filePath, const std::string& fileRoot, unsigned int fileLevel, unsigned int displayLevel, unsigned int timestampLogs); +extern bool LogInitialise(bool daemon, const std::string& filePath, const std::string& fileRoot, unsigned int fileLevel, unsigned int displayLevel); extern void LogFinalise(); #endif diff --git a/MMDVM.ini b/MMDVM.ini index 239cb94..ccc561b 100644 --- a/MMDVM.ini +++ b/MMDVM.ini @@ -26,7 +26,6 @@ DisplayLevel=1 FileLevel=1 FilePath=. FileRoot=MMDVM -TimestampLogs=1 [CW Id] Enable=1 diff --git a/MMDVMHost.cpp b/MMDVMHost.cpp index 5003cf9..f174b83 100644 --- a/MMDVMHost.cpp +++ b/MMDVMHost.cpp @@ -242,9 +242,9 @@ int CMMDVMHost::run() #endif #if !defined(_WIN32) && !defined(_WIN64) - ret = ::LogInitialise(m_daemon, m_conf.getLogFilePath(), m_conf.getLogFileRoot(), m_conf.getLogFileLevel(), m_conf.getLogDisplayLevel(), m_conf.getLogTimestampLogs()); + ret = ::LogInitialise(m_daemon, m_conf.getLogFilePath(), m_conf.getLogFileRoot(), m_conf.getLogFileLevel(), m_conf.getLogDisplayLevel()); #else - ret = ::LogInitialise(false, m_conf.getLogFilePath(), m_conf.getLogFileRoot(), m_conf.getLogFileLevel(), m_conf.getLogDisplayLevel(), m_conf.getLogTimestampLogs()); + ret = ::LogInitialise(false, m_conf.getLogFilePath(), m_conf.getLogFileRoot(), m_conf.getLogFileLevel(), m_conf.getLogDisplayLevel()); #endif if (!ret) { ::fprintf(stderr, "MMDVMHost: unable to open the log file\n"); diff --git a/RemoteCommand.cpp b/RemoteCommand.cpp index 33bd069..fd96e46 100644 --- a/RemoteCommand.cpp +++ b/RemoteCommand.cpp @@ -51,7 +51,7 @@ int main(int argc, char** argv) CRemoteCommand::CRemoteCommand(unsigned int port) : m_port(port) { - ::LogInitialise(false, ".", "RemoteCommand", 2U, 2U, 1U); + ::LogInitialise(false, ".", "RemoteCommand", 2U, 2U); } CRemoteCommand::~CRemoteCommand()