| commit | b195014e05b9d92ee1f37ab23d41c0a44d063f13 | |
| author | Hans Goudey <hans@blender.org> | |
| Thu, 4 Apr 2024 20:46:41 +0000 (4 22:46 +0200) | ||
| committer | Hans Goudey <hooglyboogly@noreply.localhost> | |
| Thu, 4 Apr 2024 20:46:41 +0000 (4 22:46 +0200) | ||
| tree | e6ca9fb6465347d357136ed6b1b3375a60fe8905 | treesnapshot (tar.gz zip) |
| parent | 23265a2b6d16709ba401ec0c2aea8c43c4ff9f64 | commitdiff |
Cleanup: Clarify modifier depends_on_normals handling
Because normals are calcualted lazily for all cases "depends_on_normals"
except for BMesh original normals, this is mostly unnecessary. It's actually
probably not necessary at all, because in practice there is always a separate
positions array stored in `EditMeshData` during mesh edit mode modifier
evaluation, bringing us back to the lazy calculation. But anyway, removing
the usage for topology-changing modifiers and modifiers which don't
accept BMesh as input anyway simplifies things.
Pull Request: https://projects.blender.org/blender/blender/pulls/120274
Because normals are calcualted lazily for all cases "depends_on_normals"
except for BMesh original normals, this is mostly unnecessary. It's actually
probably not necessary at all, because in practice there is always a separate
positions array stored in `EditMeshData` during mesh edit mode modifier
evaluation, bringing us back to the lazy calculation. But anyway, removing
the usage for topology-changing modifiers and modifiers which don't
accept BMesh as input anyway simplifies things.
Pull Request: https://projects.blender.org/blender/blender/pulls/120274
15 files changed: