More CI fixes

Chowdhury-DSP · Jun 18, 2024 · 60c4de2 · 60c4de2
1 parent d40daca
commit 60c4de2
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 14 deletions.
diff --git a/modules/common/chowdsp_data_structures/Allocators/chowdsp_STLArenaAllocator.h b/modules/common/chowdsp_data_structures/Allocators/chowdsp_STLArenaAllocator.h
@@ -28,7 +28,7 @@ struct STLArenaAllocator
 
     T* allocate (std::size_t n)
     {
-        return static_cast<T*> (arena.allocate_bytes (n, alignof (T)));
+        return static_cast<T*> (arena.allocate_bytes (n * sizeof (T), alignof (T)));
     }
 
     void deallocate (T*, std::size_t) const

diff --git a/modules/common/chowdsp_data_structures/Structures/chowdsp_AbstractTree.cpp b/modules/common/chowdsp_data_structures/Structures/chowdsp_AbstractTree.cpp
@@ -10,7 +10,7 @@ template <typename ElementType, typename DerivedType>
 AbstractTree<ElementType, DerivedType>::~AbstractTree()
 {
     doForAllNodes ([] (Node& node)
-                   { node.~Node(); });
+                   { node.leaf.reset(); });
 }
 
 template <typename ElementType, typename DerivedType>
@@ -108,7 +108,7 @@ void AbstractTree<ElementType, DerivedType>::removeNode (Node& node)
             node.parent->last_child = node.prev_sibling;
     }
 
-    node.~Node();
+    node.leaf.reset();
 }
 
 template <typename ElementType, typename DerivedType>
@@ -150,9 +150,10 @@ template <typename ElementType, typename DerivedType>
 void AbstractTree<ElementType, DerivedType>::clear()
 {
     doForAllNodes ([] (Node& node)
-                   { node.~Node(); });
+                   { node.leaf.reset(); });
     allocator.reset (64 * sizeof (Node));
     count = 0;
+    root_node = {};
 }
 
 template <typename ElementType, typename DerivedType>

diff --git a/modules/common/chowdsp_data_structures/Structures/chowdsp_EnumMap.h b/modules/common/chowdsp_data_structures/Structures/chowdsp_EnumMap.h
@@ -126,7 +126,7 @@ struct EnumMap
             {
                 ++index;
                 ++iter;
-            } while (! iter->has_value() && index < std::tuple_size_v<Storage>);
+            } while (index < std::tuple_size_v<Storage> && ! iter->has_value());
 
             if (index < std::tuple_size_v<Storage>)
                 key = magic_enum::enum_value<Key> (index);

diff --git a/tests/common_tests/chowdsp_data_structures_test/AbstractTreeTest.cpp b/tests/common_tests/chowdsp_data_structures_test/AbstractTreeTest.cpp
@@ -1,8 +1,6 @@
 #include <CatchUtils.h>
 #include <chowdsp_data_structures/chowdsp_data_structures.h>
 
-#if ! JUCE_WINDOWS // @TODO
-
 struct StringTree : chowdsp::AbstractTree<std::string, StringTree>
 {
     static Node& insert_string (std::string&& element, Node& parent_node, AbstractTree& tree)
@@ -161,4 +159,3 @@ TEST_CASE ("Abstract Tree Test", "[common][data-structures]")
             });
     }
 }
-#endif
diff --git a/tests/common_tests/chowdsp_data_structures_test/EnumMapTest.cpp b/tests/common_tests/chowdsp_data_structures_test/EnumMapTest.cpp
@@ -93,9 +93,7 @@ TEST_CASE ("Enum Map Test", "[common][data-structures]")
         iter = 0;
         for (auto [key, val] : map)
             val = static_cast<int> (iter++);
-#if ! JUCE_WINDOWS // @TODO
         REQUIRE (map[Food::Apple] == 0);
         REQUIRE (map[Food::Green_Beans] == 1);
-#endif
     }
 }
diff --git a/tests/common_tests/chowdsp_data_structures_test/STLArenaAllocatorTest.cpp b/tests/common_tests/chowdsp_data_structures_test/STLArenaAllocatorTest.cpp
@@ -1,8 +1,6 @@
 #include <CatchUtils.h>
 #include <chowdsp_data_structures/chowdsp_data_structures.h>
 
-#if ! JUCE_WINDOWS // @TODO
-
 TEST_CASE ("STL Arena Allocator Test", "[common][data-structures]")
 {
     using Arena = chowdsp::ArenaAllocator<>;
@@ -27,5 +25,3 @@ TEST_CASE ("STL Arena Allocator Test", "[common][data-structures]")
     REQUIRE (vec.size() == 5);
     REQUIRE (vec.front() == 0);
 }
-
-#endif