diff options
author | Sergey Sharybin <sergey.vfx@gmail.com> | 2019-06-06 13:24:40 +0300 |
---|---|---|
committer | Sergey Sharybin <sergey.vfx@gmail.com> | 2019-06-07 10:12:54 +0300 |
commit | 2a5fcfaad0c16a486e4582aed86b8a140ede9d40 (patch) | |
tree | 56b526d7e76284b193b0a4d6335e5392ae90887c /source/blender/blenkernel/intern/object.c | |
parent | f24a7fa31dfed0a661cb6d9fd8ebd73d42a97a40 (diff) |
Don't tag object data's bounding box as dirty before re-evaluaiton
Most of the time this actually tags evaluated mesh which will be tossed
away few lines below anyway. And this causes issues if the evaluated
mesh is actually owned by the mesh datablock.
The bounding box now has clear separation between original object data
and object data after modifiers, so this should not be causing any
issues.
Diffstat (limited to 'source/blender/blenkernel/intern/object.c')
-rw-r--r-- | source/blender/blenkernel/intern/object.c | 21 |
1 files changed, 0 insertions, 21 deletions
diff --git a/source/blender/blenkernel/intern/object.c b/source/blender/blenkernel/intern/object.c index b673c882e21..2a66edc8d42 100644 --- a/source/blender/blenkernel/intern/object.c +++ b/source/blender/blenkernel/intern/object.c @@ -429,27 +429,6 @@ static void object_update_from_subsurf_ccg(Object *object) /* free data derived from mesh, called when mesh changes or is freed */ void BKE_object_free_derived_caches(Object *ob) { - /* Also serves as signal to remake texspace. - * - * NOTE: This function can be called from threads on different objects - * sharing same data datablock. So we need to ensure atomic nature of - * data modification here. - */ - if (ob->type == OB_MESH) { - Mesh *me = ob->data; - - if (me && me->bb) { - atomic_fetch_and_or_int32(&me->bb->flag, BOUNDBOX_DIRTY); - } - } - else if (ELEM(ob->type, OB_SURF, OB_CURVE, OB_FONT)) { - Curve *cu = ob->data; - - if (cu && cu->bb) { - atomic_fetch_and_or_int32(&cu->bb->flag, BOUNDBOX_DIRTY); - } - } - MEM_SAFE_FREE(ob->runtime.bb); object_update_from_subsurf_ccg(ob); |