Release v1.7

.clang-format

@@ -26,7 +26,6 @@ BinPackArguments: true
 BinPackParameters: true
 BitFieldColonSpacing: None
 BreakBeforeBinaryOperators: NonAssignment
-BreakBeforeBraces: Custom
 BreakBeforeConceptDeclarations: true
 BreakConstructorInitializers: BeforeComma
 BreakInheritanceList: AfterComma
@@ -62,6 +61,7 @@ IndentPPDirectives: BeforeHash
 IndentRequires: true
 IndentWidth: 4
 IndentWrappedFunctionNames: false
+InsertBraces: true
 KeepEmptyLinesAtTheStartOfBlocks: false
 LambdaBodyIndentation: OuterScope
 MaxEmptyLinesToKeep: 2

.github/workflows/build.yml

@@ -12,34 +12,31 @@ on:
 
 jobs:
   Build:
-    name: ${{ matrix.os-name }} (${{ matrix.compiler }}, ${{ matrix.config }})
+    name: ${{ matrix.os }} (${{ matrix.compiler }}, ${{ matrix.config }})
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: false
       matrix:
         config: [Release, Debug]
-        name: [windows-msvc, windows-clang, linux-clang, linux-gcc, macos-clang]
-        include:
-          - name: windows-msvc
-            os: windows-latest
-            os-name: Windows
+        os:
+          - windows-2022
+          - ubuntu-24.04
+          - macos-15
+        compiler:
+          - clang-18
+          - gcc-14
+          - msvc
+        exclude:
+          - os: windows-2022
+            compiler: gcc-14
+          - os: ubuntu-24.04
             compiler: msvc
-          - name: windows-clang
-            os: windows-latest
-            os-name: Windows
-            compiler: clang-17
-          - name: linux-clang
-            os: ubuntu-latest
-            os-name: Linux
-            compiler: clang-16
-          - name: linux-gcc
-            os: ubuntu-latest
-            os-name: Linux
-            compiler: gcc-13
-          - name: macos-clang
-            os: macos-14
-            os-name: MacOS
-            compiler: clang-17
+          - os: macos-15
+            compiler: msvc
+          - os: macos-15
+            compiler: clang-18
+    env:
+      os-name: ${{ contains(matrix.os, 'windows') && 'Windows' || (contains(matrix.os, 'ubuntu') && 'Linux' || 'MacOS') }}
     steps:
       - name: Checkout repository
         uses: actions/checkout@v4
@@ -49,13 +46,14 @@ jobs:
       - name: Setup Cpp
         uses: aminya/setup-cpp@v1
         with:
-          compiler: ${{ (matrix.os-name != 'MacOS' || !contains(matrix.compiler, 'clang')) && matrix.compiler || '' }}
-          vcvarsall: ${{ matrix.os-name == 'Windows' }}
+          # Skip compiler setup for macos clang-17
+          compiler: ${{ (contains(matrix.os, 'macos') && matrix.compiler == 'clang-17') && '' || matrix.compiler }}
+          vcvarsall: ${{ contains(matrix.os, 'windows') }}
           cmake: true
           ninja: true
 
-      - name: (MacOS) Install clang through brew
-        if: matrix.os-name == 'MacOS' && contains(matrix.compiler, 'clang')
+      - name: (MacOS) Install clang-17 through brew
+        if: contains(matrix.os, 'macos') && matrix.compiler == 'clang-17'
         run: |
           brew install llvm@17
           export LLVM_DIR="$(brew --prefix llvm@17)/lib/cmake"
@@ -83,11 +81,11 @@ jobs:
         uses: actions/upload-artifact@v4
         if: ${{ contains(matrix.compiler, 'clang') }} # Only clang artifacts are stored
         with:
-          name: Pipe-${{ matrix.os-name }}-${{ matrix.config }}
+          name: Pipe-${{ env.os-name }}-${{ matrix.config }}
           path: Install
 
       - name: Run Tests
-        if: ${{ matrix.os-name != 'macos' }} # Tests are skipped temporarily on MacOS
+        if: ${{ !contains(matrix.os, 'macos') }} # Tests are skipped temporarily on MacOS
         working-directory: ./Build
         run: ctest --output-on-failure -j2 -C ${{ matrix.config }}
 

.gitignore

@@ -7,5 +7,3 @@ Saved
 /compile_commands.json
 compile_commands.json
 Install
-
-Include/Pipe/Export.h

Benchmarks/Arenas.bench.h

@@ -4,12 +4,7 @@
 #include "nanobench.h"
 using namespace ankerl;
 
-#include <Pipe/Memory/Alloc.h>
-#include <Pipe/Memory/BestFitArena.h>
-#include <Pipe/Memory/BigBestFitArena.h>
-#include <Pipe/Memory/HeapArena.h>
-#include <Pipe/Memory/MonoLinearArena.h>
-#include <Pipe/Memory/MultiLinearArena.h>
+#include <PipeMemoryArenas.h>
 #include <PipeTime.h>
 
 

Benchmarks/main.cpp

@@ -1,6 +1,6 @@
 // Copyright 2015-2024 Piperift - All rights reserved
 
-#include <Pipe/Memory/NewDelete.h>
+#include <PipeNewDelete.h>
 //  Override as first include
 
 #define ANKERL_NANOBENCH_IMPLEMENT 1

CMake/Util.cmake

@@ -91,15 +91,23 @@ function(set_option target exposure option)
 endfunction()
 
 function(target_add_link_option_if_compiles target exposure flag)
-  CHECK_CXX_COMPILER_FLAG("${flag}" COMPILER_HAS_THOSE_TOGGLES)
-  if(COMPILER_HAS_THOSE_TOGGLES)
-    target_link_options(${target_name} ${exposure} ${flag})
-  endif()
+    CHECK_CXX_COMPILER_FLAG("${flag}" COMPILER_HAS_THOSE_TOGGLES)
+    if(COMPILER_HAS_THOSE_TOGGLES)
+        target_link_options(${target_name} ${exposure} ${flag})
+    endif()
 endfunction()
 
 function(target_add_compile_option_if_compiles target exposure flag)
-  CHECK_CXX_COMPILER_FLAG("${flag}" COMPILER_HAS_THOSE_TOGGLES)
-  if(COMPILER_HAS_THOSE_TOGGLES)
-    target_compile_options(${target_name} ${exposure} ${flag})
-  endif()
+    CHECK_CXX_COMPILER_FLAG("${flag}" COMPILER_HAS_THOSE_TOGGLES)
+    if(COMPILER_HAS_THOSE_TOGGLES)
+        target_compile_options(${target_name} ${exposure} ${flag})
+    endif()
 endfunction()
+
+function(pipe_target_disable_rtti target exposure)
+    if(COMPILER_MSVC)
+        target_compile_options(${target} ${exposure} /GR-)
+    elseif(COMPILER_CLANG OR COMPILER_GCC)
+        target_compile_options(${target} ${exposure} -fno-rtti)
+    endif()
+endfunction()

CMakeLists.txt

@@ -12,7 +12,7 @@ set_property(GLOBAL PROPERTY USE_FOLDERS ON)
 ################################################################################
 # Project
 
-project(Pipe VERSION 0.1 LANGUAGES CXX)
+project(Pipe VERSION 0.1 LANGUAGES CXX C)
 
 
 if(CMAKE_PROJECT_NAME STREQUAL PROJECT_NAME)
@@ -38,7 +38,6 @@ endif()
 include(CMake/Util.cmake)
 include(CMake/LinkTimeOptimization.cmake)
 find_package(Sanitizers)
-include(GenerateExportHeader)
 
 
 # ##############################################################################
@@ -52,18 +51,17 @@ add_subdirectory(Extern)
 
 if (PIPE_BUILD_SHARED)
     add_library(Pipe SHARED)
+    target_compile_definitions(Pipe PUBLIC PIPE_IMPORTS=1)
+    target_compile_definitions(Pipe PRIVATE -DPIPE_IMPORTS PIPE_EXPORTS=1)
 else()
     add_library(Pipe STATIC)
 endif()
 
-generate_export_header(Pipe
-    BASE_NAME PIPE
-    EXPORT_MACRO_NAME PIPE_API
-    EXPORT_FILE_NAME ${CMAKE_CURRENT_SOURCE_DIR}/Include/Pipe/Export.h)
 pipe_target_define_platform(Pipe)
 
 target_include_directories(Pipe PUBLIC $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/Include>)
-file(GLOB_RECURSE PIPE_SOURCE_FILES CONFIGURE_DEPENDS Src/*.cpp Src/*.h)
+target_include_directories(Pipe PRIVATE $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/Src>)
+file(GLOB_RECURSE PIPE_SOURCE_FILES CONFIGURE_DEPENDS Src/*.cpp Src/*.c)
 target_sources(Pipe PRIVATE ${PIPE_SOURCE_FILES})
 target_compile_definitions(Pipe PRIVATE NOMINMAX)
 
@@ -75,6 +73,7 @@ pipe_target_enable_CPP20(Pipe)
 pipe_add_sanitizers(Pipe)
 pipe_target_shared_output_directory(Pipe)
 #pipe_target_link_time_optimization(Pipe)
+pipe_target_disable_rtti(Pipe PRIVATE)
 
 
 ################################################################################
@@ -130,6 +129,6 @@ install(DIRECTORY Include/Pipe
 )
 
 # Debug files
-if (PLATFORM_WINDOWS)
+if (PLATFORM_WINDOWS AND NOT COMPILER_GCC)
     install(FILES $<TARGET_PDB_FILE:Pipe> DESTINATION "${CMAKE_INSTALL_LIBDIR}" OPTIONAL)
 endif()

Include/Misc/PipeDebug.h

@@ -13,6 +13,8 @@ static_assert(false, "Imgui not found. PipeDebug requires v1.90 or newer.");
 static_assert(false, "Imgui v" IMGUI_VERSION " found but PipeDebug requires v1.90 or newer.");
 #endif
 
+// Mark debug tools as present for others
+#define P_DEBUG_TOOLS 1
 
 #include "Misc/PipeImGui.h"
 #include "Pipe/Core/Checks.h"
@@ -176,7 +178,8 @@ namespace p
 
 		EntityContext* ctx = nullptr;
 
-		bool initialized = false;
+		bool initialized  = false;
+		bool isFirstDebug = true;
 
 
 		DebugContext() = default;
@@ -871,9 +874,13 @@ namespace p
 				ImGui::PushTextColor(errorTextColor);
 				const char* errorMsg;
 				if (inspector.id == NoId)
+				{
 					errorMsg = "No entity";
+				}
 				else
+				{
 					errorMsg = removed ? "Removed entity" : "Invalid entity";
+				}
 
 				auto regionAvail = ImGui::GetContentRegionAvail();
 				auto textSize =
@@ -1139,7 +1146,9 @@ namespace p
 
 			ImGui::TableSetColumnIndex(0);    // Id
 			if (!passedFilter)
+			{
 				ImGui::PushTextColor(ImGui::GetTextColor().Shade(0.3f));
+			}
 			ImGuiTreeNodeFlags treeNodeFlags =
 			    ImGuiTreeNodeFlags_SpanAllColumns | ImGuiTreeNodeFlags_NoTreePushOnOpen;
 			if (typeProperties.IsEmpty())
@@ -1148,7 +1157,9 @@ namespace p
 			}
 			const bool propsOpen = ImGui::TreeNodeEx(idText.data(), treeNodeFlags);
 			if (!passedFilter)
+			{
 				ImGui::PopTextColor();
+			}
 
 			ImGui::TableSetColumnIndex(1);    // Name
 			StringView ns;
@@ -1277,7 +1288,8 @@ namespace p
 		    "Called EndDebug() but there was no current ECS Debug Context! Forgot "
 		    "to call "
 		    "BeginDebug()?");
-		currentContext = nullptr;
+		currentContext->isFirstDebug = false;
+		currentContext               = nullptr;
 	}
 #endif
 };    // namespace p

Include/Pipe.h

@@ -8,6 +8,6 @@
 
 namespace p
 {
-	PIPE_API void Initialize(StringView logPath = {});
-	PIPE_API void Shutdown();
+    PIPE_API void Initialize(struct Logger* logger = nullptr);
+    PIPE_API void Shutdown();
 };    // namespace p