diff options
author | Philipp Oeser <info@graphics-engineer.com> | 2020-08-27 16:47:13 +0300 |
---|---|---|
committer | Philipp Oeser <info@graphics-engineer.com> | 2020-08-27 23:12:12 +0300 |
commit | 2a0e996d772b780b67c3b3d72586f9ddae754f34 (patch) | |
tree | f5183b12028b41eb360a8510ef73de5998499409 /source/blender/editors/space_outliner/outliner_select.c | |
parent | 770cc66f75628d832fc3c3bff62057c7e9bd751a (diff) |
Fix T80159: Custom Normals Averaging crash after clearing
custom split normals data
Clearing custom split normals would get rid of the CD_CUSTOMLOOPNORMAL
layer - but editing data `lnor_spacearr` would be kept.
Adding a CD_CUSTOMLOOPNORMAL layer (if none exists yet) should be done
in `edbm_average_normals_exec` / `BKE_editmesh_lnorspace_update` /
`BM_lnorspace_update` / `BM_lnorspacearr_store`. The thing is that if
the editing data `lnor_spacearr` would still be valid after `Clear
Custom Split Normals Data`, blender would happily call
`BM_lnorspace_rebuild` instead. Doing that without a CD_CUSTOMLOOPNORMAL
layer is asking for trouble.
Now clear lnor_spacearr on `Clear Custom Split Normals Data` as well.
Thx @mont29 for feedback here.
Maniphest Tasks: T80159
Differential Revision: https://developer.blender.org/D8730
Diffstat (limited to 'source/blender/editors/space_outliner/outliner_select.c')
0 files changed, 0 insertions, 0 deletions