fixed swap_columns() fast-path breaking data()

also: - refactoring
marzer · Aug 13, 2023 · 6e1cb2d · 6e1cb2d
1 parent b314fa4
commit 6e1cb2d
Show file tree

Hide file tree

Showing 17 changed files with 1,081 additions and 906 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,7 +2,9 @@
 
 ## Unreleased
 
--   Used CRTP mixins in generated classes to reduce generated code size
+-   Fixed `swap_columns` fast-path breaking `data()`
+-   Used more CRTP mixins in generated classes
+-   Binary size improvements
 
 ## v0.5.0
 

diff --git a/examples/entities.hpp b/examples/entities.hpp
@@ -140,6 +140,11 @@ namespace soagen::detail
 		SOAGEN_MAKE_NAME(pos);
 	#endif
 
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::entities, 0, id);
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::entities, 1, name);
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::entities, 2, pos);
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::entities, 3, orient);
+
 	template <>
 	struct table_traits_type_<soagen::examples::entities>
 	{
@@ -156,24 +161,15 @@ namespace soagen::detail
 		using type = soagen::allocator;
 	};
 
-	template <>
-	struct is_soa_<soagen::examples::entities> : std::true_type
-	{};
-
-	template <>
-	struct columns_always_aligned_<soagen::examples::entities> : std::true_type
-	{};
-
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::entities, 0, id);
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::entities, 1, name);
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::entities, 2, pos);
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::entities, 3, orient);
-
 	template <>
 	struct table_type_<soagen::examples::entities>
 	{
-		using type = table<table_traits_type<soagen::examples::entities>, soagen::allocator>;
+		using type = table<table_traits_type<soagen::examples::entities>, allocator_type<soagen::examples::entities>>;
 	};
+
+	template <>
+	struct is_soa_<soagen::examples::entities> : std::true_type
+	{};
 }
 
 // clang-format on

diff --git a/examples/shapes.hpp b/examples/shapes.hpp
@@ -160,6 +160,14 @@ namespace soagen::detail
 		SOAGEN_MAKE_NAME(radius);
 	#endif
 
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::boxes, 0, center_x);
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::boxes, 1, center_y);
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::boxes, 2, center_z);
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::boxes, 3, extents_x);
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::boxes, 4, extents_y);
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::boxes, 5, extents_z);
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::boxes, 6, mass);
+
 	template <>
 	struct table_traits_type_<soagen::examples::boxes>
 	{
@@ -180,26 +188,20 @@ namespace soagen::detail
 	};
 
 	template <>
-	struct is_soa_<soagen::examples::boxes> : std::true_type
-	{};
+	struct table_type_<soagen::examples::boxes>
+	{
+		using type = table<table_traits_type<soagen::examples::boxes>, allocator_type<soagen::examples::boxes>>;
+	};
 
 	template <>
-	struct columns_always_aligned_<soagen::examples::boxes> : std::true_type
+	struct is_soa_<soagen::examples::boxes> : std::true_type
 	{};
 
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::boxes, 0, center_x);
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::boxes, 1, center_y);
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::boxes, 2, center_z);
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::boxes, 3, extents_x);
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::boxes, 4, extents_y);
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::boxes, 5, extents_z);
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::boxes, 6, mass);
-
-	template <>
-	struct table_type_<soagen::examples::boxes>
-	{
-		using type = table<table_traits_type<soagen::examples::boxes>, soagen::allocator>;
-	};
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::spheres, 0, center_x);
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::spheres, 1, center_y);
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::spheres, 2, center_z);
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::spheres, 3, radius);
+	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::spheres, 4, mass);
 
 	template <>
 	struct table_traits_type_<soagen::examples::spheres>
@@ -218,25 +220,15 @@ namespace soagen::detail
 		using type = soagen::allocator;
 	};
 
-	template <>
-	struct is_soa_<soagen::examples::spheres> : std::true_type
-	{};
-
-	template <>
-	struct columns_always_aligned_<soagen::examples::spheres> : std::true_type
-	{};
-
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::spheres, 0, center_x);
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::spheres, 1, center_y);
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::spheres, 2, center_z);
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::spheres, 3, radius);
-	SOAGEN_MAKE_NAMED_COLUMN(soagen::examples::spheres, 4, mass);
-
 	template <>
 	struct table_type_<soagen::examples::spheres>
 	{
-		using type = table<table_traits_type<soagen::examples::spheres>, soagen::allocator>;
+		using type = table<table_traits_type<soagen::examples::spheres>, allocator_type<soagen::examples::spheres>>;
 	};
+
+	template <>
+	struct is_soa_<soagen::examples::spheres> : std::true_type
+	{};
 }
 
 // clang-format on