From 2911df487ede413fb6b83b2de199f50657cc5cfe Mon Sep 17 00:00:00 2001 From: Andre Susano Pinto Date: Mon, 22 Sep 2008 22:48:44 +0000 Subject: [PATCH] Fixed shrinkwrap constraint handling of matrixs. Also since contraint uses BVHCache its now usable and fast. Being possible to shrinkwrap constraint several objects to another and only have one BVHTree being build. --- source/blender/blenkernel/intern/constraint.c | 49 ++++++++----------- 1 file changed, 20 insertions(+), 29 deletions(-) diff --git a/source/blender/blenkernel/intern/constraint.c b/source/blender/blenkernel/intern/constraint.c index db173126375..6e94fc255ac 100644 --- a/source/blender/blenkernel/intern/constraint.c +++ b/source/blender/blenkernel/intern/constraint.c @@ -3263,9 +3263,6 @@ static void shrinkwrap_get_tarmat (bConstraint *con, bConstraintOb *cob, bConstr { bShrinkwrapConstraint *scon = (bShrinkwrapConstraint *) con->data; - if (ct) - Mat4CpyMat4(ct->matrix, cob->startmat); - if( VALID_CONS_TARGET(ct) && (ct->tar->type == OB_MESH) ) { int fail = FALSE; @@ -3274,13 +3271,10 @@ static void shrinkwrap_get_tarmat (bConstraint *con, bConstraintOb *cob, bConstr float dist; SpaceTransform transform; - DerivedMesh *target = CDDM_copy( object_get_derived_final(ct->tar, CD_MASK_BAREMESH) ); + DerivedMesh *target = object_get_derived_final(ct->tar, CD_MASK_BAREMESH); BVHTreeRayHit hit; BVHTreeNearest nearest; - //TODO - //Its stupid to create a bvhtree.. if we are only going to project one vertex - //But lets do it this way.. (so that in future maybe bvhtree gets a cache system and then the tree would already be build) BVHTreeFromMesh treeData; memset( &treeData, 0, sizeof(treeData) ); @@ -3290,15 +3284,22 @@ static void shrinkwrap_get_tarmat (bConstraint *con, bConstraintOb *cob, bConstr hit.index = -1; hit.dist = 100000.0f; //TODO should use FLT_MAX.. but normal projection doenst yet supports it + Mat4One(ct->matrix); + if(target != NULL) { - - space_transform_from_matrixs(&transform, cob->startmat, ct->tar->obmat); + space_transform_from_matrixs(&transform, cob->matrix, ct->tar->obmat); switch(scon->shrinkType) { case MOD_SHRINKWRAP_NEAREST_SURFACE: - bvhtree_from_mesh_faces(&treeData, target, 0.0, 2, 6); + case MOD_SHRINKWRAP_NEAREST_VERTEX: + + if(scon->shrinkType == MOD_SHRINKWRAP_NEAREST_VERTEX) + bvhtree_from_mesh_verts(&treeData, target, 0.0, 2, 6); + else + bvhtree_from_mesh_faces(&treeData, target, 0.0, 2, 6); + if(treeData.tree == NULL) { fail = TRUE; @@ -3314,23 +3315,6 @@ static void shrinkwrap_get_tarmat (bConstraint *con, bConstraintOb *cob, bConstr space_transform_invert(&transform, co); break; - case MOD_SHRINKWRAP_NEAREST_VERTEX: - bvhtree_from_mesh_verts(&treeData, target, 0.0, 2, 6); - if(treeData.tree == NULL) - { - fail = TRUE; - break; - } - - space_transform_apply_normal(&transform, no); - - BLI_bvhtree_find_nearest(treeData.tree, co, &nearest, treeData.nearest_callback, &treeData); - - dist = VecLenf(co, nearest.co); - VecLerpf(co, co, nearest.co, (dist - scon->dist)/dist); //linear interpolation - space_transform_invert(&transform, co); - break; - case MOD_SHRINKWRAP_PROJECT: if(scon->projAxis & MOD_SHRINKWRAP_PROJECT_OVER_X_AXIS) no[0] = 1.0f; if(scon->projAxis & MOD_SHRINKWRAP_PROJECT_OVER_Y_AXIS) no[1] = 1.0f; @@ -3365,8 +3349,15 @@ static void shrinkwrap_get_tarmat (bConstraint *con, bConstraintOb *cob, bConstr target->release(target); - if(fail == FALSE) - VECADD(ct->matrix[3], ct->matrix[3], co); + if(fail == TRUE) + { + //Don't move the point + co[0] = co[1] = co[2] = 0.0f; + } + + //co is in local object coordinates, change it to global and update target position + VecMat4MulVecfl(co, cob->matrix, co); + VECCOPY(ct->matrix[3], co); } } }