diff options
| author | Lukas Tönne <lukas.toenne@gmail.com> | 2015-03-20 12:18:57 +0300 |
|---|---|---|
| committer | Lukas Tönne <lukas.toenne@gmail.com> | 2015-03-20 12:18:57 +0300 |
| commit | 229befd147a9e06202b24c1fd3e7ec8a0da59cc8 (patch) | |
| tree | 5f3169ac4f190d9a23d869a7e49c0d09e56b32a0 /source/blender/editors/space_view3d/view3d_draw.c | |
| parent | f77d75ed65ee532cf25f29d84f3d4297c8712c52 (diff) | |
Replacing boundbox temporarily during drawing is dangerous, since it
can be freed at any point.
Luckily we only need the boundbox locally for testing dupli visibility.
Diffstat (limited to 'source/blender/editors/space_view3d/view3d_draw.c')
| -rw-r--r-- | source/blender/editors/space_view3d/view3d_draw.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/source/blender/editors/space_view3d/view3d_draw.c b/source/blender/editors/space_view3d/view3d_draw.c index 689d1904952..4dccf5a1aa5 100644 --- a/source/blender/editors/space_view3d/view3d_draw.c +++ b/source/blender/editors/space_view3d/view3d_draw.c @@ -2053,11 +2053,9 @@ static void draw_dupli_objects_color( for (; dob; dob_prev = dob, dob = dob_next, dob_next = dob_next ? dupli_step(dob_next->next) : NULL) { /* for restoring after override */ DerivedMesh *store_final_dm; - BoundBox *store_bb; tbase.object = dob->ob; store_final_dm = dob->ob->derivedFinal; - store_bb = dob->ob->bb; /* Make sure lod is updated from dupli's position */ @@ -2103,7 +2101,7 @@ static void draw_dupli_objects_color( tbase.flag |= OB_FROMCACHE; tbase.object->derivedFinal = dob_data->cache_dm; - tbase.object->bb = bb_tmp = &dob_data->bb; + bb_tmp = &dob_data->bb; } } if (!bb_tmp) @@ -2181,7 +2179,6 @@ static void draw_dupli_objects_color( /* restore final DM */ tbase.object->derivedFinal = store_final_dm; - tbase.object->bb = store_bb; } if (apply_data) { |