chore: Improve logs

- decrease level of repetetive button name log - Move system reporting log to separate function
AntiMicroX · Nov 21, 2022 · 50faa0b · 50faa0b
1 parent f56ca66
commit 50faa0b
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 13 deletions.
diff --git a/src/joybuttontypes/joybutton.cpp b/src/joybuttontypes/joybutton.cpp
@@ -118,7 +118,7 @@ JoyButton::JoyButton(int sdl_button_index, int originset, SetJoystick *parentSet
     m_index_sdl = sdl_button_index;
     m_originset = originset;
     quitEvent = true;
-    VERBOSE() << "Created button with ID: " << m_index_sdl << " For set: " << originset << " Name: " << getName();
+    DEBUG() << "Created button with ID: " << m_index_sdl << " For set: " << originset << " Name: " << getName();
 }
 
 JoyButton::~JoyButton()

diff --git a/src/main.cpp b/src/main.cpp
@@ -221,6 +221,24 @@ void importLegacySettingsIfExist()
     }
 }
 
+void log_system_config()
+{
+    VERBOSE() << "AntiMicroX version: " << PadderCommon::programVersion
+#ifdef ANTIMICROX_PKG_VERSION
+              << " Package: " << ANTIMICROX_PKG_VERSION
+#endif
+#ifdef QT_DEBUG
+              << " Type: Debug"
+#else
+              << " Type: Release"
+#endif
+        ;
+    VERBOSE() << "SDL version: " << PadderCommon::sdlVersionUsed << " (Compiled with: " << PadderCommon::sdlVersionCompiled
+              << ")";
+    VERBOSE() << QString("Host OS: %1 Version: %2 Architecture: %3")
+                     .arg(QSysInfo::productType(), QSysInfo::productVersion(), QSysInfo::currentCpuArchitecture());
+}
+
 int main(int argc, char *argv[])
 {
     qInstallMessageHandler(Logger::loggerMessageHandler);
@@ -265,18 +283,6 @@ int main(int argc, char *argv[])
     }
     settings.importFromCommandLine(cmdutility);
     settings.applySettingsToLogger(cmdutility, appLogger);
-    VERBOSE() << "AntiMicroX version: " << PadderCommon::programVersion
-#ifdef ANTIMICROX_PKG_VERSION
-              << " Package: " << ANTIMICROX_PKG_VERSION
-#endif
-#ifdef QT_DEBUG
-              << " Type: Debug"
-#else
-              << " Type: Release"
-#endif
-        ;
-    VERBOSE() << "SDL version: " << PadderCommon::sdlVersionUsed << " (Compiled with: " << PadderCommon::sdlVersionCompiled
-              << ")";
 
     Q_INIT_RESOURCE(resources);
 
@@ -295,6 +301,7 @@ int main(int argc, char *argv[])
     // In the future, there might be a reason to actually send
     // messages to the QLocalServer.
     QLocalSocket socket;
+    log_system_config();
 
     if ((socket.serverName() == QString()))
     {