Small fixes and minor changes:

- Added vscode files to gitignore
- Updated logger_internal.cpp (WIN32 -> BOREALIS_WIN)
- Updated preprocessor directives in types.h
- Removed preprocessor directives around DefaultInitString test

Author: FreddyOm

.gitignore

@@ -21,3 +21,7 @@ CMakeUserPresets.json
 
 .vs/
 /CMakeSettings.json
+
+
+# VSCode files
+.vscode/

src/core/debug/logger_internal.cpp

@@ -1,15 +1,15 @@
 #include "logger_internal.h"
 #include "logger.h"
 
-#ifdef WIN32
+#ifdef BOREALIS_WIN 
 #include <Windows.h>
 #endif
 
 namespace Borealis::Debug
 {
 	Borealis::Types::int16 LogMessageInternal(const DebugInfoDesc desc)
 	{
-        #ifdef WIN32
+        #ifdef BOREALIS_WIN
 
         static HANDLE hConsole = GetStdHandle(STD_OUTPUT_HANDLE);
         switch (desc.logType)
@@ -40,7 +40,7 @@ namespace Borealis::Debug
 		const Borealis::Types::int16 result = printf("%s", desc.msg.c_str());
 		printf("\n%s, %u\n", desc.fileName.c_str(), desc.line);
 
-        #ifdef WIN32
+        #ifdef BOREALIS_WIN
         SetConsoleTextAttribute(hConsole, 15);
         #endif
 

src/core/types/types.h

@@ -5,7 +5,7 @@
 namespace Borealis::Types
 {	
 
-#ifdef BOREALIS_WIN
+#if defined(BOREALIS_WIN)
 
 		typedef int_fast8_t		int8;
 		typedef short			int16;
@@ -19,9 +19,8 @@ namespace Borealis::Types
 		typedef intptr_t		int64Ptr;
 		typedef uintptr_t		uint64Ptr;
 
-#elif BOREALIS_LINUX
+#elif defined(BOREALIS_LINUX)
 
-		// @TODO: Adapt for Linux
 		typedef int_fast8_t		int8;
 		typedef short			int16;
 		typedef int				int32;
@@ -35,7 +34,7 @@ namespace Borealis::Types
 		typedef intptr_t		int64Ptr;
 		typedef uintptr_t		uint64Ptr;
 
-#elif BOREALIS_OSX
+#elif defined(BOREALIS_OSX)
 
 		// @TODO: Adapt for OSX
 		typedef int_fast8_t		int8;
@@ -51,5 +50,9 @@ namespace Borealis::Types
 		typedef intptr_t		int64Ptr;
 		typedef uintptr_t		uint64Ptr;
 
+#else
+
+#error No valid build platform detected!
+
 #endif
 }

tests/test_string.cpp

@@ -3,8 +3,6 @@
 
 using namespace Borealis::Types;
 
-#ifdef BOREALIS_DEBUG	// Is being optimized away in RelWithDebInfo
-
 TEST(StringTest, DefaultInitString)
 {
 	StringId myString{};
@@ -13,8 +11,6 @@ TEST(StringTest, DefaultInitString)
 	EXPECT_EQ(myString, myString2);
 }
 
-#endif
-
 TEST(StringTest, StringEquality)
 {
 	StringId myString = String("MyFirstTestString");