From d9ed5f285797ea2676dddc42df9d82d29199b43b Mon Sep 17 00:00:00 2001 From: light7734 Date: Thu, 10 Jul 2025 13:38:14 +0330 Subject: [PATCH] refactor: remove stringifier --- modules/engine/CMakeLists.txt | 2 - .../include/engine/scene/components.hpp | 1 - .../include/engine/utils/stringifier.hpp | 27 ------- modules/engine/src/debug/exceptions.cpp | 5 +- modules/engine/src/utils/stringifier.cpp | 79 ------------------- 5 files changed, 2 insertions(+), 112 deletions(-) delete mode 100644 modules/engine/include/engine/utils/stringifier.hpp delete mode 100644 modules/engine/src/utils/stringifier.cpp diff --git a/modules/engine/CMakeLists.txt b/modules/engine/CMakeLists.txt index 1a98039..a970c83 100644 --- a/modules/engine/CMakeLists.txt +++ b/modules/engine/CMakeLists.txt @@ -11,7 +11,6 @@ if(NOT WIN32) scene/scene.cpp time/timer.cpp utils/serializer.cpp - utils/stringifier.cpp ) else() add_library_module(engine @@ -26,7 +25,6 @@ else() scene/scene.cpp time/timer.cpp utils/serializer.cpp - utils/stringifier.cpp ) endif() diff --git a/modules/engine/include/engine/scene/components.hpp b/modules/engine/include/engine/scene/components.hpp index a9d5793..f32366a 100644 --- a/modules/engine/include/engine/scene/components.hpp +++ b/modules/engine/include/engine/scene/components.hpp @@ -1,6 +1,5 @@ #pragma once - #include #include #include diff --git a/modules/engine/include/engine/utils/stringifier.hpp b/modules/engine/include/engine/utils/stringifier.hpp deleted file mode 100644 index d708192..0000000 --- a/modules/engine/include/engine/utils/stringifier.hpp +++ /dev/null @@ -1,27 +0,0 @@ -#pragma once -#ifndef LIGHT_STRINGIFIER_H - #define LIGHT_STRINGIFIER_H - - - -namespace Light { - -enum class GraphicsAPI; - -class Stringifier -{ -public: - static auto glDebugMsgSeverity(unsigned int severity) -> std::string; - - static auto glDebugMsgSource(unsigned int source) -> std::string; - - static auto glDebugMsgType(unsigned int type) -> std::string; - - static auto spdlogLevel(unsigned int level) -> std::string; - - static auto graphics_api_to_string(GraphicsAPI api) -> std::string; -}; - -} // namespace Light - -#endif diff --git a/modules/engine/src/debug/exceptions.cpp b/modules/engine/src/debug/exceptions.cpp index 138274f..00d9845 100644 --- a/modules/engine/src/debug/exceptions.cpp +++ b/modules/engine/src/debug/exceptions.cpp @@ -1,5 +1,4 @@ #include -#include #include #ifdef LIGHT_PLATFORM_WINDOWS @@ -20,8 +19,8 @@ glException::glException(unsigned int source, unsigned int type, unsigned int id log_crt("glException::glException::"); // log_crt(" Severity: {}", // Stringifier::glDebugMsgSeverity(GL_DEBUG_SEVERITY_HIGH)); - log_crt(" Source : {}", Stringifier::glDebugMsgSource(source)); - log_crt(" Type : {}", Stringifier::glDebugMsgType(type)); + // log_crt(" Source : {}", Stringifier::glDebugMsgSource(source)); + // log_crt(" Type : {}", Stringifier::glDebugMsgType(type)); log_crt(" ID : {}", id); // log_crt(" Vendor : {}", glGetString(GL_VENDOR)); // log_crt(" renderer: {}", glGetString(GL_RENDERER)); diff --git a/modules/engine/src/utils/stringifier.cpp b/modules/engine/src/utils/stringifier.cpp deleted file mode 100644 index 92c3a92..0000000 --- a/modules/engine/src/utils/stringifier.cpp +++ /dev/null @@ -1,79 +0,0 @@ -#include -#include -#include -#include - -namespace Light { - -auto Stringifier::glDebugMsgSeverity(unsigned int severity) -> std::string -{ - switch (severity) - { - case GL_DEBUG_SEVERITY_NOTIFICATION: return "GL_DEBUG_SEVERITY_NOTIFICATION"; - case GL_DEBUG_SEVERITY_LOW: return "GL_DEBUG_SEVERITY_LOW"; - case GL_DEBUG_SEVERITY_MEDIUM: return "GL_DEBUG_SEVERITY_MEDIUM"; - case GL_DEBUG_SEVERITY_HIGH: return "GL_DEBUG_SEVERITY_HIGH"; - default: return "UNKNOWN"; - } -} - -auto Stringifier::glDebugMsgSource(unsigned int source) -> std::string -{ - switch (source) - { - case GL_DEBUG_SOURCE_API: return "GL_DEBUG_SOURCE_API"; - case GL_DEBUG_SOURCE_WINDOW_SYSTEM: return "GL_DEBUG_SOURCE_WINDOW_SYSTEM"; - case GL_DEBUG_SOURCE_SHADER_COMPILER: return "GL_DEBUG_SOURCE_SHADER_COMPILER"; - case GL_DEBUG_SOURCE_THIRD_PARTY: return "GL_DEBUG_SOURCE_THIRD_PARTY"; - case GL_DEBUG_SOURCE_APPLICATION: return "GL_DEBUG_SOURCE_APPLICATION"; - case GL_DEBUG_SOURCE_OTHER: return "GL_DEBUG_SOURCE_OTHER"; - default: return "UNKNOWN"; - } -} - -auto Stringifier::glDebugMsgType(unsigned int type) -> std::string -{ - switch (type) - { - case GL_DEBUG_TYPE_ERROR: return "GL_DEBUG_TYPE_ERROR"; - case GL_DEBUG_TYPE_DEPRECATED_BEHAVIOR: return "GL_DEBUG_TYPE_DEPRECATED_BEHAVIOR"; - case GL_DEBUG_TYPE_UNDEFINED_BEHAVIOR: return "GL_DEBUG_TYPE_UNDEFINED_BEHAVIOR"; - case GL_DEBUG_TYPE_PORTABILITY: return "GL_DEBUG_TYPE_PORTABILITY"; - case GL_DEBUG_TYPE_PERFORMANCE: return "GL_DEBUG_TYPE_PERFORMANCE"; - case GL_DEBUG_TYPE_MARKER: return "GL_DEBUG_TYPE_MARKER"; - case GL_DEBUG_TYPE_PUSH_GROUP: return "GL_DEBUG_TYPE_PUSH_GROUP"; - case GL_DEBUG_TYPE_POP_GROUP: return "GL_DEBUG_TYPE_POP_GROUP"; - case GL_DEBUG_TYPE_OTHER: return "GL_DEBUG_TYPE_OTHER"; - default: return "UNKNOWN"; - } -} - -auto Stringifier::spdlogLevel(unsigned int level) -> std::string -{ - switch (level) - { - case SPDLOG_LEVEL_TRACE: return "SPDLOG_LEVEL_TRACE"; - case SPDLOG_LEVEL_DEBUG: return "SPDLOG_LEVEL_DEBUG"; - case SPDLOG_LEVEL_INFO: return "SPDLOG_LEVEL_INFO"; - case SPDLOG_LEVEL_WARN: return "SPDLOG_LEVEL_WARN"; - case SPDLOG_LEVEL_ERROR: return "SPDLOG_LEVEL_ERROR"; - case SPDLOG_LEVEL_CRITICAL: return "SPDLOG_LEVEL_CRITICAL"; - case SPDLOG_LEVEL_OFF: return "SPDLOG_LEVEL_OFF"; - default: return "UNKNOWN"; - } -} - -auto Stringifier::graphics_api_to_string(GraphicsAPI api) -> std::string -{ - switch (api) - { - case Light::GraphicsAPI::Default: return "Default"; - case Light::GraphicsAPI::OpenGL: return "OpenGL"; - case Light::GraphicsAPI::DirectX: return "DirectX"; - case Light::GraphicsAPI::Vulkan: return "Vulkan"; - case Light::GraphicsAPI::Metal: return "Metal"; - default: return "UNKNOWN"; - } -} - -} // namespace Light