#839 Ensure on_add hooks are invoked when using bulk_init

Author: SanderMertens

examples/c/systems/system_ctx/project.json

@@ -6,5 +6,10 @@
             "flecs"
         ],
         "public": false
+    },
+    "lang.c": {
+        "${os linux}": {
+            "lib": ["m"]
+        }
     }
-}
+}

examples/c/systems/time_interval/src/main.c

@@ -1,5 +1,4 @@
 #include <time_interval.h>
-#include <basics.h>
 #include <stdio.h>
 
 // This example shows how to run a system at a specified time interval.

examples/cpp/systems/system_ctx/project.json

@@ -7,5 +7,10 @@
         ],
         "public": false,
         "language": "c++"
+    },
+    "lang.cpp": {
+        "${os linux}": {
+            "lib": ["m"]
+        }
     }
-}
+}

examples/cpp/systems/system_ctx/src/main.cpp

@@ -33,10 +33,10 @@ int main(int, char *[]) {
     auto sys = ecs.system<const Position, const Radius>("Collide")
         .ctx(&q_collide)
         .each([](flecs::iter& it, size_t i, const Position& p1, const Radius& r1) {
-            CollisionQuery *q_collide = it.ctx<CollisionQuery>();
+            CollisionQuery *w = it.ctx<CollisionQuery>();
             flecs::entity e1 = it.entity(i);
 
-            q_collide->each([&](flecs::entity e2, const Position& p2, const Radius& r2) {
+            q->each([&](flecs::entity e2, const Position& p2, const Radius& r2) {
                 if (e1 == e2) {
                     // don't collide with self
                     return;

flecs.c

@@ -3819,9 +3819,7 @@ int32_t flecs_table_grow_data(
     /* Initialize entity ids and record ptrs */
     int32_t i;
     if (ids) {
-        for (i = 0; i < to_add; i ++) {
-            e[i] = ids[i];
-        }
+        ecs_os_memcpy_n(e, ids, ecs_entity_t, to_add);
     } else {
         ecs_os_memset(e, 0, ECS_SIZEOF(ecs_entity_t) * to_add);
     }
@@ -3834,6 +3832,8 @@ int32_t flecs_table_grow_data(
         ecs_type_info_t *ti = type_info[i];
         flecs_table_grow_column(world, column, ti, to_add, size, true);
         ecs_assert(columns[i].size == size, ECS_INTERNAL_ERROR, NULL);
+        flecs_run_add_hooks(world, table, ti, column, e, table->type.array[i], 
+            cur_count, to_add, false);
     }
 
     /* Add elements to each switch column */
@@ -4303,11 +4303,11 @@ int32_t flecs_table_appendn(
     ecs_assert(!table->lock, ECS_LOCKED_STORAGE, NULL);
 
     flecs_table_check_sanity(table);
-
     int32_t cur_count = flecs_table_data_count(data);
     int32_t result = flecs_table_grow_data(
         world, table, data, to_add, cur_count + to_add, ids);
     flecs_table_check_sanity(table);
+
     return result;
 }
 

src/table.c

@@ -1438,9 +1438,7 @@ int32_t flecs_table_grow_data(
     /* Initialize entity ids and record ptrs */
     int32_t i;
     if (ids) {
-        for (i = 0; i < to_add; i ++) {
-            e[i] = ids[i];
-        }
+        ecs_os_memcpy_n(e, ids, ecs_entity_t, to_add);
     } else {
         ecs_os_memset(e, 0, ECS_SIZEOF(ecs_entity_t) * to_add);
     }
@@ -1453,6 +1451,8 @@ int32_t flecs_table_grow_data(
         ecs_type_info_t *ti = type_info[i];
         flecs_table_grow_column(world, column, ti, to_add, size, true);
         ecs_assert(columns[i].size == size, ECS_INTERNAL_ERROR, NULL);
+        flecs_run_add_hooks(world, table, ti, column, e, table->type.array[i], 
+            cur_count, to_add, false);
     }
 
     /* Add elements to each switch column */
@@ -1922,11 +1922,11 @@ int32_t flecs_table_appendn(
     ecs_assert(!table->lock, ECS_LOCKED_STORAGE, NULL);
 
     flecs_table_check_sanity(table);
-
     int32_t cur_count = flecs_table_data_count(data);
     int32_t result = flecs_table_grow_data(
         world, table, data, to_add, cur_count + to_add, ids);
     flecs_table_check_sanity(table);
+
     return result;
 }
 

test/api/project.json

@@ -815,6 +815,7 @@
                 "ctor_on_move_pair",
                 "move_on_realloc",
                 "move_on_bulk_new",
+                "on_add_on_bulk_new",
                 "move_on_delete",
                 "move_dtor_on_delete",
                 "copy_on_override_pair",