[Bf-blender-cvs] [bdb5852e28f] master: Cleanup: Remove mesh valid assertions in debug builds

Hans Goudey noreply at git.blender.org
Mon Dec 20 19:47:15 CET 2021


Commit: bdb5852e28f90a8574da725fff2cf023a583dbb8
Author: Hans Goudey
Date:   Mon Dec 20 12:46:33 2021 -0600
Branches: master
https://developer.blender.org/rBbdb5852e28f90a8574da725fff2cf023a583dbb8

Cleanup: Remove mesh valid assertions in debug builds

These are useful for development, but when the primitive nodes
aren't actively changing, the performance cost is not worth it.

===================================================================

M	source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_circle.cc
M	source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_grid.cc
M	source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_uv_sphere.cc

===================================================================

diff --git a/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_circle.cc b/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_circle.cc
index 8239b72c5ad..503915872d6 100644
--- a/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_circle.cc
+++ b/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_circle.cc
@@ -213,8 +213,6 @@ static void node_geo_exec(GeoNodeExecParams params)
 
   Mesh *mesh = create_circle_mesh(radius, verts_num, fill);
 
-  BLI_assert(BKE_mesh_is_valid(mesh));
-
   params.set_output("Mesh", GeometrySet::create_with_mesh(mesh));
 }
 
diff --git a/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_grid.cc b/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_grid.cc
index e4e52c3cdc7..4a5668453bf 100644
--- a/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_grid.cc
+++ b/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_grid.cc
@@ -212,7 +212,6 @@ static void node_geo_exec(GeoNodeExecParams params)
   }
 
   Mesh *mesh = create_grid_mesh(verts_x, verts_y, size_x, size_y);
-  BLI_assert(BKE_mesh_is_valid(mesh));
   BKE_id_material_eval_ensure_default_slot(&mesh->id);
 
   params.set_output("Mesh", GeometrySet::create_with_mesh(mesh));
diff --git a/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_uv_sphere.cc b/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_uv_sphere.cc
index ce2e0923a30..f3213f791de 100644
--- a/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_uv_sphere.cc
+++ b/source/blender/nodes/geometry/nodes/node_geo_mesh_primitive_uv_sphere.cc
@@ -287,8 +287,6 @@ static Mesh *create_uv_sphere_mesh(const float radius, const int segments, const
 
   calculate_sphere_uvs(mesh, segments, rings);
 
-  BLI_assert(BKE_mesh_is_valid(mesh));
-
   return mesh;
 }



More information about the Bf-blender-cvs mailing list