[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [44331] trunk/blender/source/blender/ blenkernel/intern/armature.c: Style code cleanup and consistancy ( mainly spaces around assignements, C++ comments to C, and misc spaces/ tabs cleanup).
Bastien Montagne
montagne29 at wanadoo.fr
Wed Feb 22 16:35:50 CET 2012
Revision: 44331
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=44331
Author: mont29
Date: 2012-02-22 15:35:42 +0000 (Wed, 22 Feb 2012)
Log Message:
-----------
Style code cleanup and consistancy (mainly spaces around assignements, C++ comments to C, and misc spaces/tabs cleanup). No functional change.
Modified Paths:
--------------
trunk/blender/source/blender/blenkernel/intern/armature.c
Modified: trunk/blender/source/blender/blenkernel/intern/armature.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/armature.c 2012-02-22 15:04:22 UTC (rev 44330)
+++ trunk/blender/source/blender/blenkernel/intern/armature.c 2012-02-22 15:35:42 UTC (rev 44331)
@@ -4,7 +4,7 @@
* This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version 2
- * of the License, or (at your option) any later version.
+ * of the License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ -73,84 +73,87 @@
#include "BIK_api.h"
#include "BKE_sketch.h"
-/* **************** Generic Functions, data level *************** */
+/* **************** Generic Functions, data level *************** */
bArmature *add_armature(const char *name)
{
bArmature *arm;
-
- arm= alloc_libblock (&G.main->armature, ID_AR, name);
+
+ arm = alloc_libblock (&G.main->armature, ID_AR, name);
arm->deformflag = ARM_DEF_VGROUP|ARM_DEF_ENVELOPE;
arm->flag = ARM_COL_CUSTOM; /* custom bone-group colors */
- arm->layer= 1;
+ arm->layer = 1;
return arm;
}
bArmature *get_armature(Object *ob)
{
- if(ob->type==OB_ARMATURE)
+ if(ob->type == OB_ARMATURE)
return (bArmature *)ob->data;
return NULL;
}
-void free_bonelist (ListBase *lb)
+void free_bonelist(ListBase *lb)
{
Bone *bone;
- for(bone=lb->first; bone; bone=bone->next) {
+ for(bone = lb->first; bone; bone = bone->next) {
if(bone->prop) {
IDP_FreeProperty(bone->prop);
MEM_freeN(bone->prop);
}
free_bonelist(&bone->childbase);
}
-
+
BLI_freelistN(lb);
}
void free_armature(bArmature *arm)
{
- if (arm) {
+ if(arm) {
free_bonelist(&arm->bonebase);
-
+
/* free editmode data */
- if (arm->edbo) {
+ if(arm->edbo) {
BLI_freelistN(arm->edbo);
-
+
MEM_freeN(arm->edbo);
- arm->edbo= NULL;
+ arm->edbo = NULL;
}
/* free sketch */
- if (arm->sketch) {
+ if(arm->sketch) {
freeSketch(arm->sketch);
arm->sketch = NULL;
}
-
+
/* free animation data */
- if (arm->adt) {
+ if(arm->adt) {
BKE_free_animdata(&arm->id);
- arm->adt= NULL;
+ arm->adt = NULL;
}
}
}
void make_local_armature(bArmature *arm)
{
- Main *bmain= G.main;
- int is_local= FALSE, is_lib= FALSE;
+ Main *bmain = G.main;
+ int is_local = FALSE, is_lib = FALSE;
Object *ob;
- if (arm->id.lib==NULL) return;
- if (arm->id.us==1) {
+ if(arm->id.lib == NULL)
+ return;
+ if(arm->id.us == 1) {
id_clear_lib_data(bmain, &arm->id);
return;
}
- for(ob= bmain->object.first; ob && ELEM(0, is_lib, is_local); ob= ob->id.next) {
+ for(ob = bmain->object.first; ob && ELEM(0, is_lib, is_local); ob = ob->id.next) {
if(ob->data == arm) {
- if(ob->id.lib) is_lib= TRUE;
- else is_local= TRUE;
+ if(ob->id.lib)
+ is_lib = TRUE;
+ else
+ is_local = TRUE;
}
}
@@ -158,16 +161,16 @@
id_clear_lib_data(bmain, &arm->id);
}
else if(is_local && is_lib) {
- bArmature *arm_new= copy_armature(arm);
- arm_new->id.us= 0;
+ bArmature *arm_new = copy_armature(arm);
+ arm_new->id.us = 0;
/* Remap paths of new ID using old library as base. */
BKE_id_lib_local_paths(bmain, arm->id.lib, &arm_new->id);
- for(ob= bmain->object.first; ob; ob= ob->id.next) {
+ for(ob = bmain->object.first; ob; ob = ob->id.next) {
if(ob->data == arm) {
- if(ob->id.lib==NULL) {
- ob->data= arm_new;
+ if(ob->id.lib == NULL) {
+ ob->data = arm_new;
arm_new->id.us++;
arm->id.us--;
}
@@ -176,105 +179,104 @@
}
}
-static void copy_bonechildren (Bone* newBone, Bone* oldBone, Bone* actBone, Bone **newActBone)
+static void copy_bonechildren(Bone* newBone, Bone* oldBone, Bone* actBone, Bone **newActBone)
{
- Bone *curBone, *newChildBone;
-
+ Bone *curBone, *newChildBone;
+
if(oldBone == actBone)
- *newActBone= newBone;
+ *newActBone = newBone;
if(oldBone->prop)
- newBone->prop= IDP_CopyProperty(oldBone->prop);
+ newBone->prop = IDP_CopyProperty(oldBone->prop);
- /* Copy this bone's list*/
+ /* Copy this bone's list */
BLI_duplicatelist(&newBone->childbase, &oldBone->childbase);
-
- /* For each child in the list, update it's children*/
- newChildBone=newBone->childbase.first;
- for (curBone=oldBone->childbase.first;curBone;curBone=curBone->next){
- newChildBone->parent=newBone;
+
+ /* For each child in the list, update it's children */
+ newChildBone = newBone->childbase.first;
+ for(curBone = oldBone->childbase.first; curBone; curBone = curBone->next){
+ newChildBone->parent = newBone;
copy_bonechildren(newChildBone, curBone, actBone, newActBone);
- newChildBone=newChildBone->next;
+ newChildBone = newChildBone->next;
}
}
bArmature *copy_armature(bArmature *arm)
{
bArmature *newArm;
- Bone *oldBone, *newBone;
- Bone *newActBone= NULL;
-
- newArm= copy_libblock(&arm->id);
+ Bone *oldBone, *newBone;
+ Bone *newActBone= NULL;
+
+ newArm = copy_libblock(&arm->id);
BLI_duplicatelist(&newArm->bonebase, &arm->bonebase);
-
- /* Duplicate the childrens' lists*/
- newBone=newArm->bonebase.first;
- for (oldBone=arm->bonebase.first;oldBone;oldBone=oldBone->next){
- newBone->parent=NULL;
- copy_bonechildren (newBone, oldBone, arm->act_bone, &newActBone);
- newBone=newBone->next;
- };
-
- newArm->act_bone= newActBone;
- newArm->edbo= NULL;
- newArm->act_edbone= NULL;
- newArm->sketch= NULL;
+ /* Duplicate the childrens' lists*/
+ newBone = newArm->bonebase.first;
+ for(oldBone = arm->bonebase.first; oldBone; oldBone = oldBone->next) {
+ newBone->parent = NULL;
+ copy_bonechildren(newBone, oldBone, arm->act_bone, &newActBone);
+ newBone = newBone->next;
+ }
+ newArm->act_bone = newActBone;
+
+ newArm->edbo = NULL;
+ newArm->act_edbone = NULL;
+ newArm->sketch = NULL;
+
return newArm;
}
-static Bone *get_named_bone_bonechildren (Bone *bone, const char *name)
+static Bone *get_named_bone_bonechildren(Bone *bone, const char *name)
{
Bone *curBone, *rbone;
-
- if (!strcmp (bone->name, name))
+
+ if(!strcmp(bone->name, name))
return bone;
-
- for (curBone=bone->childbase.first; curBone; curBone=curBone->next){
- rbone=get_named_bone_bonechildren (curBone, name);
- if (rbone)
+
+ for(curBone = bone->childbase.first; curBone; curBone = curBone->next) {
+ rbone = get_named_bone_bonechildren(curBone, name);
+ if(rbone)
return rbone;
}
-
+
return NULL;
}
-Bone *get_named_bone (bArmature *arm, const char *name)
-/*
- Walk the list until the bone is found
- */
+/* Walk the list until the bone is found */
+Bone *get_named_bone(bArmature *arm, const char *name)
{
- Bone *bone=NULL, *curBone;
-
- if (!arm) return NULL;
-
- for (curBone=arm->bonebase.first; curBone; curBone=curBone->next){
- bone = get_named_bone_bonechildren (curBone, name);
- if (bone)
+ Bone *bone = NULL, *curBone;
+
+ if(!arm)
+ return NULL;
+
+ for(curBone = arm->bonebase.first; curBone; curBone = curBone->next) {
+ bone = get_named_bone_bonechildren(curBone, name);
+ if(bone)
return bone;
}
-
+
return bone;
}
-/* Finds the best possible extension to the name on a particular axis. (For renaming, check for unique names afterwards)
- * strip_number: removes number extensions (TODO: not used)
- * axis: the axis to name on
- * head/tail: the head/tail co-ordinate of the bone on the specified axis
- */
-int bone_autoside_name (char name[MAXBONENAME], int UNUSED(strip_number), short axis, float head, float tail)
+/* Finds the best possible extension to the name on a particular axis. (For renaming, check for
+ * unique names afterwards) strip_number: removes number extensions (TODO: not used)
+ * axis: the axis to name on
+ * head/tail: the head/tail co-ordinate of the bone on the specified axis */
+int bone_autoside_name(char name[MAXBONENAME], int UNUSED(strip_number), short axis, float head, float tail)
{
unsigned int len;
- char basename[MAXBONENAME]= "";
- char extension[5]= "";
+ char basename[MAXBONENAME] = "";
+ char extension[5] = "";
- len= strlen(name);
- if (len == 0) return 0;
+ len = strlen(name);
+ if(len == 0)
+ return 0;
BLI_strncpy(basename, name, sizeof(basename));
-
- /* Figure out extension to append:
+
+ /* Figure out extension to append:
* - The extension to append is based upon the axis that we are working on.
* - If head happens to be on 0, then we must consider the tail position as well to decide
* which side the bone is on
@@ -282,31 +284,31 @@
* -> Otherwise, extension is added from perspective of object based on which side tail goes to
* - If head is non-zero, extension is added from perspective of object based on side head is on
*/
- if (axis == 2) {
+ if(axis == 2) {
/* z-axis - vertical (top/bottom) */
- if (IS_EQ(head, 0)) {
- if (tail < 0)
+ if(IS_EQ(head, 0)) {
+ if(tail < 0)
strcpy(extension, "Bot");
- else if (tail > 0)
+ else if(tail > 0)
strcpy(extension, "Top");
}
else {
- if (head < 0)
+ if(head < 0)
strcpy(extension, "Bot");
else
strcpy(extension, "Top");
}
}
- else if (axis == 1) {
+ else if(axis == 1) {
/* y-axis - depth (front/back) */
- if (IS_EQ(head, 0)) {
- if (tail < 0)
+ if(IS_EQ(head, 0)) {
+ if(tail < 0)
strcpy(extension, "Fr");
- else if (tail > 0)
+ else if(tail > 0)
strcpy(extension, "Bk");
}
else {
- if (head < 0)
+ if(head < 0)
strcpy(extension, "Fr");
else
strcpy(extension, "Bk");
@@ -314,55 +316,58 @@
}
else {
/* x-axis - horizontal (left/right) */
- if (IS_EQ(head, 0)) {
- if (tail < 0)
+ if(IS_EQ(head, 0)) {
+ if(tail < 0)
strcpy(extension, "R");
- else if (tail > 0)
+ else if(tail > 0)
strcpy(extension, "L");
}
else {
- if (head < 0)
+ if(head < 0)
strcpy(extension, "R");
- else if (head > 0)
+ /* XXX Shouldn't this be simple else, as for z and y axes? */
+ else if(head > 0)
strcpy(extension, "L");
}
}
- /* Simple name truncation
+ /* Simple name truncation
* - truncate if there is an extension and it wouldn't be able to fit
* - otherwise, just append to end
*/
- if (extension[0]) {
+ if(extension[0]) {
int change = 1;
-
- while (change) { /* remove extensions */
+
+ while(change) { /* remove extensions */
change = 0;
- if (len > 2 && basename[len-2]=='.') {
@@ Diff output truncated at 10240 characters. @@
More information about the Bf-blender-cvs
mailing list