Blender Git Commit Log

Git Commits -> Revision 527ddb0

Revision 527ddb0 by Sergey Sharybin (master)
August 19, 2013, 09:25 (GMT)
Move bevel list and path from Curve to Object datablock

I know this is not so much nice to have this guys hanging
around in a general Object datablock and ideally they better
be wrapped around into a structure like DerivedMesh or
something like this. But this is pure runtime only stuff and
we could re-wrap them around later.

Main purpose of this is making curves more thread safe,
so no separate threads will ever start freeing the same path
or the same bevel list.

It also makes sense because path and bevel shall include
deformation coming from modifiers which are applying on
pre-tesselation point and different objects could have
different set of modifiers. This used to be really confusing
in the past and now data which depends on object is stored
in an object, making things clear for understanding even.

This doesn't make curve code fully thread-safe due to
pre-tesselation modifiers still modifies actual nurbs and
lock is still needed in makeDispListsCurveTypes, but this
change makes usage of paths safe for threading.

Once modifiers will stop modifying actual nurbs, curves
will be fully safe for threading.

Actually, this commit also contains wrapping runtime curve
members into own structure

This allows easier assignment on file loading, keeps curve-
specific runtime data grouped and saves couple of bytes in
Object for non-curve types.

--
svn merge -r57938:57939 ^/branches/soc-2013-depsgraph_mt
svn merge -r57957:57958^/branches/soc-2013-depsgraph_mt

Commit Details:

Full Hash: 527ddb0a5bcb9cd0b132a449b40987f70817bb31
SVN Revision: 59265
Parent Commit: 0312b18
Lines Changed: +197, -129

25 Modified Paths:

/source/blender/blenkernel/BKE_curve.h (+7, -0) (Diff)
/source/blender/blenkernel/BKE_object.h (+1, -0) (Diff)
/source/blender/blenkernel/intern/anim.c (+7, -7) (Diff)
/source/blender/blenkernel/intern/armature.c (+3, -6) (Diff)
/source/blender/blenkernel/intern/cdderivedmesh.c (+7, -1) (Diff)
/source/blender/blenkernel/intern/constraint.c (+8, -14) (Diff)
/source/blender/blenkernel/intern/curve.c (+18, -21) (Diff)
/source/blender/blenkernel/intern/depsgraph.c (+1, -0) (Diff)
/source/blender/blenkernel/intern/displist.c (+23, -13) (Diff)
/source/blender/blenkernel/intern/effect.c (+3, -2) (Diff)
/source/blender/blenkernel/intern/font.c (+5, -3) (Diff)
/source/blender/blenkernel/intern/lattice.c (+16, -11) (Diff)
/source/blender/blenkernel/intern/material.c (+3, -1) (Diff)
/source/blender/blenkernel/intern/mball.c (+2, -1) (Diff)
/source/blender/blenkernel/intern/mesh.c (+13, -2) (Diff)
/source/blender/blenkernel/intern/object.c (+34, -10) (Diff)
/source/blender/blenloader/intern/readfile.c (+3, -4) (Diff)
/source/blender/editors/object/object_add.c (+7, -7) (Diff)
/source/blender/editors/space_info/info_stats.c (+2, -2) (Diff)
/source/blender/editors/space_view3d/drawobject.c (+19, -14) (Diff)
/source/blender/editors/space_view3d/view3d_iterators.c (+1, -1) (Diff)
/source/blender/makesdna/DNA_curve_types.h (+0, -3) (Diff)
/source/blender/makesdna/DNA_object_types.h (+3, -1) (Diff)
/source/blender/makesrna/intern/rna_main_api.c (+7, -2) (Diff)
/source/blender/modifiers/intern/MOD_array.c (+4, -3) (Diff)
By: Miika HämäläinenLast update: Nov-07-2014 14:18MiikaHweb | 2003-2021