From fe3c41f857c044542b96c8b5e0805fc2b008836f Mon Sep 17 00:00:00 2001
From: sithlord48 <sithlord48@gmail.com>
Date: Tue, 8 Jul 2025 08:32:29 -0400
Subject: [PATCH] refactor: remove Unneed enum for LogOutputters and use a
 constexp instead

---
 src/lib/base/LogOutputters.cpp | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/lib/base/LogOutputters.cpp b/src/lib/base/LogOutputters.cpp
index 7d4b2ca29..8a588ae1e 100644
--- a/src/lib/base/LogOutputters.cpp
+++ b/src/lib/base/LogOutputters.cpp
@@ -14,10 +14,7 @@
 #include <fstream>
 #include <iostream>
 
-enum EFileLogOutputter
-{
-  kFileSizeLimit = 1024 // kb
-};
+constexpr auto s_logFileSizeLimit = 1024 * 1024; //!< Max Log size before rotating (1Mb)
 
 //
 // StopLogOutputter
@@ -154,7 +151,7 @@ bool FileLogOutputter::write(LogLevel level, const char *message)
 
     // when file size exceeds limits, move to 'old log' filename.
     size_t p = m_handle.tellp();
-    if (p > (kFileSizeLimit * 1024)) {
+    if (p > s_logFileSizeLimit) {
       moveFile = true;
     }
   }