Differential D658 Diff 2130 source/gameengine/Converter/BL_BlenderDataConversion.cpp

Changeset View

Standalone View

source/gameengine/Converter/BL_BlenderDataConversion.cpp

Context not available.

	static std::map<int, SCA_IInputDevice::KX_EnumInputs> gReverseKeyTranslateTable = create_translate_table();	static std::map<int, SCA_IInputDevice::KX_EnumInputs> gReverseKeyTranslateTable = create_translate_table();


		void setup_constraint4object(KX_GameObject* obj_src, KX_Scene* scene, bRigidBodyJointConstraint* dat, PHY_IPhysicsController* phy_src, PHY_IPhysicsController* phy_dest)
		{
		/* we need to pass a full constraint frame, not just axis */
		MT_Matrix3x3 localCFrame(MT_Vector3(dat->axX,dat->axY,dat->axZ));
		MT_Vector3 axis0 = localCFrame.getColumn(0);
		MT_Vector3 axis1 = localCFrame.getColumn(1);
		MT_Vector3 axis2 = localCFrame.getColumn(2);
		MT_Vector3 scale = obj_src->NodeGetWorldScaling();

		/* apply not only the pivot and axis values, but also take scale into count
		* this is not working well, if only one or two axis are scaled, but works ok on
		* homogeneous scaling */
		int constraintId = scene->GetPhysicsEnvironment()->CreateConstraint(phy_src, phy_dest,
		(PHY_ConstraintType)dat->type,
		(float)(dat->pivX*scale.x()),
		(float)(dat->pivY*scale.y()),
		(float)(dat->pivZ*scale.z()),
		(float)(axis0.x()*scale.x()),
		(float)(axis0.y()*scale.y()),
		(float)(axis0.z()*scale.z()),
		(float)(axis1.x()*scale.x()),
		(float)(axis1.y()*scale.y()),
		(float)(axis1.z()*scale.z()),
		(float)(axis2.x()*scale.x()),
		(float)(axis2.y()*scale.y()),
		(float)(axis2.z()*scale.z()),
		dat->flag);
		/* PHY_POINT2POINT_CONSTRAINT=1,
		* PHY_LINEHINGE_CONSTRAINT=2,
		* PHY_ANGULAR_CONSTRAINT = 3,
		* PHY_CONE_TWIST_CONSTRAINT = 4,
		* PHY_VEHICLE_CONSTRAINT=11,
		* PHY_GENERIC_6DOF_CONSTRAINT=12 */

		if (constraintId) {
		/* if it is a generic 6DOF constraint, set all the limits accordingly */
		if (dat->type == PHY_GENERIC_6DOF_CONSTRAINT) {
		int dof;
		int dofbit=1;
		for (dof=0;dof<6;dof++) {
		if (dat->flag & dofbit) {
		scene->GetPhysicsEnvironment()->SetConstraintParam(constraintId, dof, dat->minLimit[dof], dat->maxLimit[dof]);
		}
		else {
		/* minLimit > maxLimit means free(disabled limit) for this degree of freedom */
		scene->GetPhysicsEnvironment()->SetConstraintParam(constraintId, dof, 1 ,-1);
		}
		dofbit<<=1;
		}
		}

		else if (dat->type == PHY_CONE_TWIST_CONSTRAINT) {
		int dof;
		int dofbit = 1<<3; /* bit flag use_angular_limit_x */
		for (dof=3;dof<6;dof++) {
		if(dat->flag & dofbit) {
		scene->GetPhysicsEnvironment()->SetConstraintParam(constraintId, dof, dat->minLimit[dof], dat->maxLimit[dof]);
		}
		else {
		/* maxLimit < 0 means free(disabled limit) for this degree of freedom */
		scene->GetPhysicsEnvironment()->SetConstraintParam(constraintId,dof,1,-1);
		}
		dofbit<<=1;
		}
		}

		else if (dat->type == PHY_LINEHINGE_CONSTRAINT) {
		int dof = 3; /* dof for angular x */
		int dofbit = 1<<3; /* bit flag use_angular_limit_x */

		if (dat->flag & dofbit) {
		scene->GetPhysicsEnvironment()->SetConstraintParam(constraintId, dof, dat->minLimit[dof], dat->maxLimit[dof]);
		}
		else {
		/* minLimit > maxLimit means free(disabled limit) for this degree of freedom */
		scene->GetPhysicsEnvironment()->SetConstraintParam(constraintId,dof,1,-1);
		}
		}

		ConstraintData *constraint = new ConstraintData(dat->type, constraintId, scene->GetPhysicsEnvironment());
		obj_src->AddConstraintData(constraint);
		}
		}

	static unsigned int KX_rgbaint2uint_new(unsigned int icol)	static unsigned int KX_rgbaint2uint_new(unsigned int icol)
	{	{
	union	union
		sybrenUnsubmitted Not Done Inline Actions To me the name "findConstraint" suggests it finds a constraint and returns it. This seems more like an "isConstraintInList" kind of function. Also it needs a comment to describe its intended use. sybren: To me the name "findConstraint" suggests it finds a constraint and returns it. This seems more…
		sybrenUnsubmitted Not Done Inline Actions Please add a comment that explains in which situations it is possible that a constraint would be converted twice, as this won't be obvious to everybody. You've invested time to understand this topic -- ensure that your knowledge of why things are done the way they are done is saved for future developers (including future you) ;-) sybren: Please add a comment that explains in which situations it is possible that a constraint would…
Context not available.
	// is not in a separate thread.	// is not in a separate thread.
	BL_Texture::GetMaxUnits();	BL_Texture::GetMaxUnits();

		/* we have to ensure that group definitions are only converted once
		* push all converted group members to this set */
		set<KX_GameObject*> convertedlist;

	if (alwaysUseExpandFraming) {	if (alwaysUseExpandFraming) {
	frame_type = RAS_FrameSettings::e_frame_extend;	frame_type = RAS_FrameSettings::e_frame_extend;
	aspect_width = canvas->GetWidth();	aspect_width = canvas->GetWidth();
Context not available.
	rendertools,	rendertools,
	converter,	converter,
	libloading);	libloading);

		/* insert object to the constraint game object list
		* so we can check later if there is a instance in the scene or
		* an instance and its actual group definition */
		convertedlist.insert((KX_GameObject*)gameobj->AddRef());

	// this code is copied from above except that	// this code is copied from above except that
	// object from groups are never in active layer	// object from groups are never in active layer
		sybrenUnsubmitted Not Done Inline Actions This could be replaced with `if (curcon->type != CONSTRAINT_TYPE_RIGIDBODYJOINT) continue;` to unindent the rest of the code block. sybren: This could be replaced with `if (curcon->type != CONSTRAINT_TYPE_RIGIDBODYJOINT) continue;` to…
		sybrenUnsubmitted Not Done Inline Actions The body of this `if` seems complicated enough to warrant its own method. Setting `skip = true` and then testing for that, it would simply be `return` when a separate method is used. sybren: The body of this `if` seems complicated enough to warrant its own method. Setting `skip = true`…
		sybrenUnsubmitted Not Done Inline Actions Style: Spaces around `=` and after commas. sybren: Style: Spaces around `=` and after commas.
		sybrenUnsubmitted Not Done Inline Actions `((gotar->GetLayer()&activeLayerBitInfo) != 0)` can be written as `(gotar->GetLayer() & activeLayerBitInfo)` sybren: `((gotar->GetLayer()&activeLayerBitInfo) != 0)` can be written as `(gotar->GetLayer() &…
		sybrenUnsubmitted Not Done Inline Actions This can be moved outside the loop. sybren: This can be moved outside the loop.
		sybrenUnsubmitted Not Done Inline Actions Style: spaces around `=` and `!=`. sybren: Style: spaces around `=` and `!=`.
		sybrenUnsubmitted Not Done Inline Actions Why can these cases be skipped? Maybe add a comment that explains this. sybren: Why can these cases be skipped? Maybe add a comment that explains this.
		sybrenUnsubmitted Not Done Inline Actions Skipp has one "p" ;-) sybren: Skipp has one "p" ;-)
Context not available.
	bConstraint *curcon;	bConstraint *curcon;
	conlist = get_active_constraints2(blenderobject);	conlist = get_active_constraints2(blenderobject);

	if ((gameobj->GetLayer()&activeLayerBitInfo)==0)
	continue;

	if (conlist) {	if (conlist) {
	for (curcon = (bConstraint )conlist->first; curcon; curcon = (bConstraint )curcon->next) {	for (curcon = (bConstraint )conlist->first; curcon; curcon = (bConstraint )curcon->next) {
	if (curcon->type==CONSTRAINT_TYPE_RIGIDBODYJOINT) {	if (curcon->type==CONSTRAINT_TYPE_RIGIDBODYJOINT) {

	bRigidBodyJointConstraint dat=(bRigidBodyJointConstraint )curcon->data;	bRigidBodyJointConstraint dat = (bRigidBodyJointConstraint )curcon->data;

	if (!dat->child && !(curcon->flag & CONSTRAINT_OFF)) {	/* to get grouped objects and instances in one active layer to work with constraints */
		if (dat->tar)
	PHY_IPhysicsController* physctr2 = 0;	gameobj->AddConstraint(dat);

	if (dat->tar)	if (!dat->child && !(curcon->flag & CONSTRAINT_OFF)) {
	{	/* prevents blender to crash, object was already converted or constraint
	KX_GameObject *gotar=getGameOb(dat->tar->id.name+2,sumolist);	* is added to game object for later replication */
	if (gotar && ((gotar->GetLayer()&activeLayerBitInfo)!=0) && gotar->GetPhysicsController())	set<KX_GameObject*>::iterator gobit;
	physctr2 = gotar->GetPhysicsController();	bool skip = false;
		for (gobit=convertedlist.begin(); gobit != convertedlist.end(); gobit++) {
		if((*gobit)->GetName() == gameobj->GetName()) {
		skip = true;
		break;
		}
	}	}

		if(skip)
		continue;

	if (gameobj->GetPhysicsController())	if (dat->tar)
	{	{
	PHY_IPhysicsController* physctrl = gameobj->GetPhysicsController();	KX_GameObject *gotar = getGameOb(dat->tar->id.name+2,sumolist);
	//we need to pass a full constraint frame, not just axis	if (gotar && ((gotar->GetLayer()&activeLayerBitInfo) != 0) && gotar->GetPhysicsController()
		&& ((gameobj->GetLayer()&activeLayerBitInfo) != 0) && gameobj->GetPhysicsController())
	//localConstraintFrameBasis
	MT_Matrix3x3 localCFrame(MT_Vector3(dat->axX,dat->axY,dat->axZ));
	MT_Vector3 axis0 = localCFrame.getColumn(0);
	MT_Vector3 axis1 = localCFrame.getColumn(1);
	MT_Vector3 axis2 = localCFrame.getColumn(2);

	int constraintId = kxscene->GetPhysicsEnvironment()->CreateConstraint(physctrl,physctr2,(PHY_ConstraintType)dat->type,(float)dat->pivX,
	(float)dat->pivY,(float)dat->pivZ,
	(float)axis0.x(),(float)axis0.y(),(float)axis0.z(),
	(float)axis1.x(),(float)axis1.y(),(float)axis1.z(),
	(float)axis2.x(),(float)axis2.y(),(float)axis2.z(),dat->flag);
	if (constraintId)
	{	{
	//if it is a generic 6DOF constraint, set all the limits accordingly	PHY_IPhysicsController *physctrl = gameobj->GetPhysicsController();
	if (dat->type == PHY_GENERIC_6DOF_CONSTRAINT)	PHY_IPhysicsController *physctr2 = gotar->GetPhysicsController();
	{	setup_constraint4object(gameobj, kxscene, dat, physctrl, physctr2);
	int dof;
	int dofbit=1;
	for (dof=0;dof<6;dof++)
	{
	if (dat->flag & dofbit)
	{
	kxscene->GetPhysicsEnvironment()->SetConstraintParam(constraintId,dof,dat->minLimit[dof],dat->maxLimit[dof]);
	} else
	{
	//minLimit > maxLimit means free(disabled limit) for this degree of freedom
	kxscene->GetPhysicsEnvironment()->SetConstraintParam(constraintId,dof,1,-1);
	}
	dofbit<<=1;
	}
	}
	else if (dat->type == PHY_CONE_TWIST_CONSTRAINT)
	{
	int dof;
	int dofbit = 1<<3; // bitflag use_angular_limit_x

	for (dof=3;dof<6;dof++)
	{
	if (dat->flag & dofbit)
	{
	kxscene->GetPhysicsEnvironment()->SetConstraintParam(constraintId,dof,dat->minLimit[dof],dat->maxLimit[dof]);
	}
	else
	{
	//maxLimit < 0 means free(disabled limit) for this degree of freedom
	kxscene->GetPhysicsEnvironment()->SetConstraintParam(constraintId,dof,1,-1);
	}
	dofbit<<=1;
	}
	}
	else if (dat->type == PHY_LINEHINGE_CONSTRAINT)
	{
	int dof = 3; // dof for angular x
	int dofbit = 1<<3; // bitflag use_angular_limit_x

	if (dat->flag & dofbit)
	{
	kxscene->GetPhysicsEnvironment()->SetConstraintParam(constraintId,dof,
	dat->minLimit[dof],dat->maxLimit[dof]);
	} else
	{
	//minLimit > maxLimit means free(disabled limit) for this degree of freedom
	kxscene->GetPhysicsEnvironment()->SetConstraintParam(constraintId,dof,1,-1);
	}
	}
	}	}
	}	}
	}	}
Context not available.
	}	}
	}	}

		/* cleanup converted set of group objects */
		set<KX_GameObject*>::iterator gobit;
		for (gobit=convertedlist.begin(); gobit!=convertedlist.end(); gobit++)
		(*gobit)->Release();

		convertedlist.clear();
	sumolist->Release();	sumolist->Release();

	// convert world	// convert world
Context not available.