From 82188ae9000f64689ccf3274e0633184e18b158f Mon Sep 17 00:00:00 2001
From: johndoe6345789 <john.doe6345789@gmail.com>
Date: Sun, 4 Jan 2026 14:17:35 +0000
Subject: [PATCH] feat: Refactor logging in ApplicationController and
 RenderController for consistency

---
 src/controllers/application_controller.cpp | 8 ++++----
 src/controllers/render_controller.cpp      | 4 ++--
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/controllers/application_controller.cpp b/src/controllers/application_controller.cpp
index 2ecd76d..9044c70 100644
--- a/src/controllers/application_controller.cpp
+++ b/src/controllers/application_controller.cpp
@@ -21,7 +21,7 @@ ApplicationController::~ApplicationController() {
 }
 
 void ApplicationController::Run() {
-    logger_->Trace("ApplicationController::Run: Entering");
+    logger_->Trace("ApplicationController", "Run", "", "Entering");
     logger_->Info("ApplicationController::Run: ApplicationController::Run starting");
     logger_->Info("ApplicationController::Run: Application starting main loop");
 
@@ -48,7 +48,7 @@ void ApplicationController::Run() {
     }
 
     logger_->Info("ApplicationController::Run: Application exiting main loop");
-    logger_->Trace("ApplicationController::Run: Exiting");
+    logger_->Trace("ApplicationController", "Run", "", "Exiting");
 }
 
 void ApplicationController::HandleEvents() {
@@ -72,7 +72,7 @@ void ApplicationController::HandleEvents() {
 }
 
 void ApplicationController::ProcessFrame(float deltaTime) {
-    logger_->Trace("ApplicationController::ProcessFrame: Entering");
+    logger_->Trace("ApplicationController", "ProcessFrame", "deltaTime=" + std::to_string(deltaTime), "Entering");
 
     // Update physics
     auto physicsService = registry_.GetService<services::IPhysicsService>();
@@ -92,7 +92,7 @@ void ApplicationController::ProcessFrame(float deltaTime) {
     // renderController->RenderFrame(static_cast<float>(std::chrono::duration_cast<std::chrono::milliseconds>(
     //     std::chrono::high_resolution_clock::now().time_since_epoch()).count()) / 1000.0f);
 
-    logger_->Trace("ApplicationController::ProcessFrame: Exiting");
+    logger_->Trace("ApplicationController", "ProcessFrame", "", "Exiting");
 }
 
 }  // namespace sdl3cpp::controllers
diff --git a/src/controllers/render_controller.cpp b/src/controllers/render_controller.cpp
index 7b5307b..0c7eb27 100644
--- a/src/controllers/render_controller.cpp
+++ b/src/controllers/render_controller.cpp
@@ -9,11 +9,11 @@ namespace sdl3cpp::controllers {
 
 RenderController::RenderController(di::ServiceRegistry& registry)
     : registry_(registry), logger_(registry.GetService<services::ILogger>()) {
-    logger_->Trace("RenderController::RenderController: Created");
+    logger_->Trace("RenderController", "RenderController", "", "Created");
 }
 
 RenderController::~RenderController() {
-    logger_->Trace("RenderController::~RenderController: Destroyed");
+    logger_->Trace("RenderController", "~RenderController", "", "Destroyed");
 }
 
 void RenderController::RenderFrame(float time) {