Differential D5258 Diff 16389 source/blender/blenkernel/intern/action.c

Changeset View

Standalone View

source/blender/blenkernel/intern/action.c

Context not available.
	}	}
	}	}

		static void pose_channels_remove_internal_links(Object ob, bPoseChannel target_pchan)
		sergeyUnsubmitted Not Done Inline Actions Don't think `target` is best name here. sergey: Don't think `target` is best name here.
		fclemAuthorUnsubmitted Not Done Inline Actions What would be a better name? Does `unlinked_pchan`sounds better? fclem: What would be a better name? Does `unlinked_pchan`sounds better?
		sergeyUnsubmitted Not Done Inline Actions Sounds good to me. sergey: Sounds good to me.
		{
		bPoseChannel pchan, pchan_next;
		bConstraint *con;
		sergeyUnsubmitted Not Done Inline Actions Is this needed? sergey: Is this needed?

		for (pchan = ob->pose->chanbase.first; pchan; pchan = pchan_next) {
		pchan_next = pchan->next;

		if (pchan->bbone_prev == target_pchan) {
		pchan->bbone_prev = NULL;
		}
		if (pchan->bbone_next == target_pchan) {
		pchan->bbone_next = NULL;
		}
		sergeyUnsubmitted Not Done Inline Actions Not entirely sure how the bbones are stored internally, but usually in the linked list you would do something like: pchan->bbone_prev = target_pchar->bbone_prev Or maybe even: target_pchar->bbone_prev->bbone_next = target_pchar->bbone_next; target_pchar->bbone_next->bbone_prev = target_pchar->bbone_prevt; But i am not sure whether bbone chain thingamajig is supposed to "break" on removing pchan or not. Not even sure if pchan can be part of multiple bbone chains. sergey: Not entirely sure how the bbones are stored internally, but usually in the linked list you…
		brechtUnsubmitted Not Done Inline Actions It's not a linked list I think, just a cache to quickly lookup the next/prev bone for b-bones. So I don't think this part is an issue. brecht: It's not a linked list I think, just a cache to quickly lookup the next/prev bone for b-bones.
		sergeyUnsubmitted Not Done Inline Actions Ok, then ignore my comment :) sergey: Ok, then ignore my comment :)
		if (pchan->custom_tx == target_pchan) {
		pchan->custom_tx = NULL;
		}
		}
		}

	/**	/**
	* Selectively remove pose channels.	* Selectively remove pose channels.
	*/	*/
Context not available.
	if (filter_fn(pchan->name, user_data)) {	if (filter_fn(pchan->name, user_data)) {
	/* Bone itself is being removed */	/* Bone itself is being removed */
	BKE_pose_channel_free(pchan);	BKE_pose_channel_free(pchan);
		pose_channels_remove_internal_links(ob, pchan);
	if (ob->pose->chanhash) {	if (ob->pose->chanhash) {
	BLI_ghash_remove(ob->pose->chanhash, pchan->name, NULL, NULL);	BLI_ghash_remove(ob->pose->chanhash, pchan->name, NULL, NULL);
	}	}
Context not available.