diff options
author | Bastien Montagne <mont29> | 2020-02-05 17:49:57 +0300 |
---|---|---|
committer | Bastien Montagne <b.mont29@gmail.com> | 2020-02-05 17:51:56 +0300 |
commit | 06c560864607562acdc1e61b13369f24b22c102c (patch) | |
tree | deedb8f06ba521b3187071b138dfdb07cd7e2dae /source/blender/blenkernel/intern/mesh_convert.c | |
parent | ea2e6995a9083e1dff0b46909397cba2c639884e (diff) |
Readfile: Proper fix for `recalc` flags clearing in non-real IDs
In current `readfile.c` code we have that:
https://developer.blender.org/diffusion/B/browse/master/source/blender/blenloader/intern/readfile.c$3523
This is unconditionally clearing nodetree's recalc flags, and was added by rB81a762e79f83 ages ago. Thing is, in main ID read code we only clear that flag when **not** in undo context.
This proposed change intends to properly handle those cases, by moving `id.recalc` flags clearing from `read_libblock()` to `direct_link_id()`, which is also called for all 'local' IDs (ntrees and master collections currently).
I’d expect that change to be straightforward (and maybe even fixing some odd undocumented bugs), however there is no .blend file testcases associated with changes in rB81a762e79f83, so wouldn’t mind that to be double checked before it goes to master.
Reviewed By: brecht
Differential Revision: https://developer.blender.org/D6711
Diffstat (limited to 'source/blender/blenkernel/intern/mesh_convert.c')
0 files changed, 0 insertions, 0 deletions