Change the fatal method in Logger

src/System/Application.cpp

@@ -23,7 +23,7 @@ void Application::onWindowClose(WindowCloseEvent *const event)
 Application::Application(ContainerBuilder *containerBuilder)
     : containerBuilder(containerBuilder), running(true), onWindowCloseHandle(0)
 {
-    logger.logInformation("Welcome to the Bare rendering framework");
+    logger.logInformation("Bare is warming up");
 
     // Register default dependency injection modules
     containerBuilder->registerType<Dispatch>().as<IDispatch>().singleInstance();
@@ -55,6 +55,8 @@ void Application::initialize()
 
     // Initialize the window
     window->initialize("Bare", 720, AspectRatio(16, 9), 1.2f);
+
+    logger.logInformation("Ready! - Userland awaits you");
 }
 
 void Application::run()

src/System/Display/SdlWindow.cpp

@@ -55,8 +55,7 @@ void SdlWindow::initialize(const string &title, int size, AspectRatio aspect, fl
         auto error = string(SDL_GetError());
         auto errorMessage = "SDL window could not be created: " + error;
 
-        logger.logError(errorMessage);
-        throw Exception(errorMessage);
+        logger.logFatal(errorMessage);
     }
 
     SDL_SetWindowMinimumSize(windowHandle, width, height);
@@ -68,11 +67,10 @@ void SdlWindow::initialize(const string &title, int size, AspectRatio aspect, fl
         auto error = string(SDL_GetError());
         auto errorMessage = "SDL window manager info could not be created: " + error;
 
-        logger.logError(errorMessage);
-        throw Exception(errorMessage);
+        logger.logFatal(errorMessage);
     }
 
-    SdlContextHandle* contextHandle;
+    const SdlContextHandle* contextHandle;
     switch (wmInfo.subsystem) {
         case SDL_SYSWM_COCOA:
             contextHandle = new SdlContextHandle(nullptr, wmInfo.info.cocoa.window);

src/System/Logging/Logger.hpp

@@ -10,6 +10,9 @@
 #include <fmt/color.h>
 #include <boost/core/demangle.hpp>
 
+#include "System/Exception.hpp"
+
+using Bare::System::Exception;
 using fmt::format;
 using fmt::format_args;
 using fmt::make_format_args;
@@ -121,7 +124,9 @@ void Logger<T>::logFatal(const string &message, const Args &... args)
 {
 #if LOG_LEVEL <= LEVEL_FATAL
     log(CHANNEL_FATAL, message, make_format_args(args...));
-    exit(1);
+
+    // TODO: Evaluate if this solution makes sense
+    throw Exception(message);
 #endif
 }
 

src/System/Renderer/BgfxRenderer.cpp

@@ -34,9 +34,10 @@ void BgfxRenderer::initialize(const IContextHandle &contextHandle)
     platformData.backBufferDS = nullptr;
 
     Init initParams;
+
     initParams.platformData = platformData;
-    init(initParams);
 
+    init(initParams);
     setDebug(BGFX_DEBUG_TEXT);
 
     logger.logInformation("Successfully initialized bgfx renderer");