Fix: Remove incorrect assert in mesh modifier evaluation
Since we have a node that sets a mesh's auto smooth angle
(unfortunately, in retrospect), we generally can't assume at all
that value is the same as whatever input mesh. Similar asserts
were removed previously in 8216b759e9
. While the attempt
at assertions to clarify assumptions is noble, this one doesn't
make sense anymore.
I found this while investigating T95479.
Differential Revision: https://developer.blender.org/D14009
This commit is contained in:
parent
a5be935966
commit
0b4cf2984f
Notes:
blender-bot
2023-02-14 06:37:09 +01:00
Referenced by issue #94932, Blender 3.0 fails to load, stuck on reading from the user prefs
|
@ -1731,13 +1731,6 @@ static void editbmesh_calc_modifiers(struct Depsgraph *depsgraph,
|
|||
BKE_id_free(nullptr, mesh_orco);
|
||||
}
|
||||
|
||||
/* Ensure normals calculation below is correct (normal settings have transferred properly).
|
||||
* However, nodes modifiers might create meshes from scratch or transfer meshes from other
|
||||
* objects with different settings, and in general it doesn't make sense to guarantee that
|
||||
* the settings are the same as the original mesh. If necessary, this could become a modifier
|
||||
* type flag. */
|
||||
BLI_assert(mesh_input->smoothresh == mesh_cage->smoothresh);
|
||||
|
||||
/* Compute normals. */
|
||||
editbmesh_calc_modifier_final_normals(mesh_final, &final_datamask);
|
||||
if (mesh_cage && (mesh_cage != mesh_final)) {
|
||||
|
|
Loading…
Reference in New Issue