refactor: merge memory_used.hpp and system_version.hpp into system_info.hpp

include/Cubed/tools/memory_used.hpp

@@ -1,29 +0,0 @@
-#pragma once
-#ifdef _WIN32
-#define WIN32_LEAN_AND_MEAN
-#include <windows.h>
-#include <psapi.h>
-#else
-#include <sys/resource.h>
-#include <unistd.h>
-#include <fstream>
-#endif
-
-inline size_t get_current_rss() {
-#ifdef _WIN32
-    PROCESS_MEMORY_COUNTERS_EX pmc;
-    if (GetProcessMemoryInfo(GetCurrentProcess(), (PROCESS_MEMORY_COUNTERS*)&pmc, sizeof(pmc))) {
-        return pmc.WorkingSetSize;
-    }
-    return 0;
-#elif defined(__linux__)
-    std::ifstream statm("/proc/self/statm");
-    long vsz = 0, rss_pages = 0;
-    statm >> vsz >> rss_pages;
-    statm.close();
-    long page_size = sysconf(_SC_PAGESIZE);
-    return rss_pages * page_size;
-#else
-    return 0; // Unsupported platform
-#endif
-}

include/Cubed/tools/system_info.hpp

@@ -6,8 +6,18 @@
 #ifdef _WIN32
 #define WIN32_LEAN_AND_MEAN
 #include <windows.h>
+#include <psapi.h>
 typedef LONG (WINAPI* RtlGetVersionPtr)(PRTL_OSVERSIONINFOW);
+#elif defined(__linux__)
+#include <sys/resource.h>
+#include <unistd.h>
+#include <fstream>
+#endif
+
+namespace Tools {
+
 inline bool get_os_version(std::string& str) {
+#ifdef _WIN32
     HMODULE hntdll = GetModuleHandleW(L"ntdll.dll");
     if (!hntdll) return false;
 
@@ -28,12 +38,7 @@ inline bool get_os_version(std::string& str) {
         str = "Windows Build " + std::to_string(osvi.dwBuildNumber);
     }
     return true;
-}
-
 #elif defined(__linux__)
-#include <fstream>
-
-inline bool get_os_version(std::string& str) {
     std::ifstream file("/etc/os-release");
     if (!file.is_open()) {
         Logger::error("Can't Open /etc/os-release");
@@ -57,12 +62,29 @@ inline bool get_os_version(std::string& str) {
         }
     }
     return false;
-}
 #else
-inline bool get_os_version(std::string& str) {
     str = "Unknown OS";
     return false;
-}
 #endif
+}
 
+inline size_t get_current_rss() {
+#ifdef _WIN32
+    PROCESS_MEMORY_COUNTERS_EX pmc;
+    if (GetProcessMemoryInfo(GetCurrentProcess(), (PROCESS_MEMORY_COUNTERS*)&pmc, sizeof(pmc))) {
+        return pmc.WorkingSetSize;
+    }
+    return 0;
+#elif defined(__linux__)
+    std::ifstream statm("/proc/self/statm");
+    long vsz = 0, rss_pages = 0;
+    statm >> vsz >> rss_pages;
+    statm.close();
+    long page_size = sysconf(_SC_PAGESIZE);
+    return rss_pages * page_size;
+#else
+    return 0; // Unsupported platform
+#endif
+}
 
+}

src/app.cpp

@@ -4,7 +4,7 @@
 #include <Cubed/map_table.hpp>
 #include <Cubed/tools/cubed_assert.hpp>
 #include <Cubed/tools/log.hpp>
-#include <Cubed/tools/memory_used.hpp>
+#include <Cubed/tools/system_info.hpp>
 #include <Cubed/tools/perlin_noise.hpp>
 
 #include <exception>
@@ -166,7 +166,7 @@ void App::update() {
         frame_count = 0;
         fps_time_count = 0.0f;
         DebugCollector::get().report("fps", std::string{"FPS: " + std::to_string(fps)});
-        DebugCollector::get().report("rss", std::format("RSS: {}mb", get_current_rss() / (1024 * 1024)));
+        DebugCollector::get().report("rss", std::format("RSS: {}mb", Tools::get_current_rss() / (1024 * 1024)));
     }
     m_texture_manager.update();
     m_world.update(delta_time);

src/debug_collector.cpp

@@ -1,6 +1,6 @@
 #include <Cubed/debug_collector.hpp>
 
-#include <Cubed/tools/system_version.hpp>
+#include <Cubed/tools/system_info.hpp>
 #include <Cubed/tools/cubed_hash.hpp>
 
 DebugCollector::DebugCollector() {
@@ -44,7 +44,7 @@ void DebugCollector::init_text() {
         os_text
         .scale(0.8f)
         .position(0.0f, 250.0f);
-    if (get_os_version(os)) {
+    if (Tools::get_os_version(os)) {
         os_text
             .text("OS: " + os);
         Logger::info("System: {}", os);