diff options
| author | Hans Goudey <h.goudey@me.com> | 2022-01-22 01:04:16 +0300 |
|---|---|---|
| committer | Hans Goudey <h.goudey@me.com> | 2022-01-22 01:04:16 +0300 |
| commit | 1c23a067069463190b084af9560d4fe6263e6fe2 (patch) | |
| tree | 0906f15a7a902672173ba9e37079351f503da48e /source/blender/editors | |
| parent | e07b217669e227eda80d1f7b341c875a2d05dc5e (diff) | |
Fix: Applying object transform can create normal layers
Because this operator is used on original objects, it's best to tag
the normals dirty instead of explicitly calculating them, to avoid
unnecessarily storing normal layers on an original object (since
they might have to be recalculated during evaluation anyway).
There may be other places this change is helpful, but being
conservative is likely better for now.
Related to T95125
Diffstat (limited to 'source/blender/editors')
| -rw-r--r-- | source/blender/editors/object/object_transform.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/source/blender/editors/object/object_transform.c b/source/blender/editors/object/object_transform.c index c320313643d..6d04bd3f9d5 100644 --- a/source/blender/editors/object/object_transform.c +++ b/source/blender/editors/object/object_transform.c @@ -811,8 +811,8 @@ static int apply_objects_internal(bContext *C, /* adjust data */ BKE_mesh_transform(me, mat, true); - /* update normals */ - BKE_mesh_calc_normals(me); + /* If normal layers exist, they are now dirty. */ + BKE_mesh_normals_tag_dirty(me); } else if (ob->type == OB_ARMATURE) { bArmature *arm = ob->data; |