[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [45125] trunk/blender/source/blender: style cleanup: follow style guide for formatting of if/for/while loops, and else if's

Campbell Barton ideasman42 at gmail.com
Sat Mar 24 07:18:51 CET 2012


Revision: 45125
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=45125
Author:   campbellbarton
Date:     2012-03-24 06:18:31 +0000 (Sat, 24 Mar 2012)
Log Message:
-----------
style cleanup: follow style guide for formatting of if/for/while loops, and else if's

Modified Paths:
--------------
    trunk/blender/source/blender/blenkernel/intern/DerivedMesh.c
    trunk/blender/source/blender/blenkernel/intern/action.c
    trunk/blender/source/blender/blenkernel/intern/anim.c
    trunk/blender/source/blender/blenkernel/intern/anim_sys.c
    trunk/blender/source/blender/blenkernel/intern/blender.c
    trunk/blender/source/blender/blenkernel/intern/bmfont.c
    trunk/blender/source/blender/blenkernel/intern/boids.c
    trunk/blender/source/blender/blenkernel/intern/booleanops_mesh.c
    trunk/blender/source/blender/blenkernel/intern/brush.c
    trunk/blender/source/blender/blenkernel/intern/bvhutils.c
    trunk/blender/source/blender/blenkernel/intern/camera.c
    trunk/blender/source/blender/blenkernel/intern/cdderivedmesh.c
    trunk/blender/source/blender/blenkernel/intern/cloth.c
    trunk/blender/source/blender/blenkernel/intern/collision.c
    trunk/blender/source/blender/blenkernel/intern/colortools.c
    trunk/blender/source/blender/blenkernel/intern/constraint.c
    trunk/blender/source/blender/blenkernel/intern/context.c
    trunk/blender/source/blender/blenkernel/intern/curve.c
    trunk/blender/source/blender/blenkernel/intern/customdata.c
    trunk/blender/source/blender/blenkernel/intern/customdata_file.c
    trunk/blender/source/blender/blenkernel/intern/depsgraph.c
    trunk/blender/source/blender/blenkernel/intern/displist.c
    trunk/blender/source/blender/blenkernel/intern/dynamicpaint.c
    trunk/blender/source/blender/blenkernel/intern/effect.c
    trunk/blender/source/blender/blenkernel/intern/fcurve.c
    trunk/blender/source/blender/blenkernel/intern/fluidsim.c
    trunk/blender/source/blender/blenkernel/intern/fmodifier.c
    trunk/blender/source/blender/blenkernel/intern/font.c
    trunk/blender/source/blender/blenkernel/intern/group.c
    trunk/blender/source/blender/blenkernel/intern/icons.c
    trunk/blender/source/blender/blenkernel/intern/idprop.c
    trunk/blender/source/blender/blenkernel/intern/image.c
    trunk/blender/source/blender/blenkernel/intern/image_gen.c
    trunk/blender/source/blender/blenkernel/intern/implicit.c
    trunk/blender/source/blender/blenkernel/intern/key.c
    trunk/blender/source/blender/blenkernel/intern/lamp.c
    trunk/blender/source/blender/blenkernel/intern/lattice.c
    trunk/blender/source/blender/blenkernel/intern/library.c
    trunk/blender/source/blender/blenkernel/intern/material.c
    trunk/blender/source/blender/blenkernel/intern/mball.c
    trunk/blender/source/blender/blenkernel/intern/mesh.c
    trunk/blender/source/blender/blenkernel/intern/mesh_validate.c
    trunk/blender/source/blender/blenkernel/intern/modifier.c
    trunk/blender/source/blender/blenkernel/intern/movieclip.c
    trunk/blender/source/blender/blenkernel/intern/multires.c
    trunk/blender/source/blender/blenkernel/intern/navmesh_conversion.c
    trunk/blender/source/blender/blenkernel/intern/node.c
    trunk/blender/source/blender/blenkernel/intern/object.c
    trunk/blender/source/blender/blenkernel/intern/ocean.c
    trunk/blender/source/blender/blenkernel/intern/packedFile.c
    trunk/blender/source/blender/blenkernel/intern/paint.c
    trunk/blender/source/blender/blenkernel/intern/particle.c
    trunk/blender/source/blender/blenkernel/intern/particle_system.c
    trunk/blender/source/blender/blenkernel/intern/pointcache.c
    trunk/blender/source/blender/blenkernel/intern/property.c
    trunk/blender/source/blender/blenkernel/intern/report.c
    trunk/blender/source/blender/blenkernel/intern/sca.c
    trunk/blender/source/blender/blenkernel/intern/screen.c
    trunk/blender/source/blender/blenkernel/intern/seqcache.c
    trunk/blender/source/blender/blenkernel/intern/seqeffects.c
    trunk/blender/source/blender/blenkernel/intern/sequencer.c
    trunk/blender/source/blender/blenkernel/intern/shrinkwrap.c
    trunk/blender/source/blender/blenkernel/intern/sketch.c
    trunk/blender/source/blender/blenkernel/intern/smoke.c
    trunk/blender/source/blender/blenkernel/intern/softbody.c
    trunk/blender/source/blender/blenkernel/intern/sound.c
    trunk/blender/source/blender/blenkernel/intern/speaker.c
    trunk/blender/source/blender/blenkernel/intern/subsurf_ccg.c
    trunk/blender/source/blender/blenkernel/intern/suggestions.c
    trunk/blender/source/blender/blenkernel/intern/text.c
    trunk/blender/source/blender/blenkernel/intern/texture.c
    trunk/blender/source/blender/blenkernel/intern/tracking.c
    trunk/blender/source/blender/blenkernel/intern/world.c
    trunk/blender/source/blender/blenkernel/intern/writeavi.c
    trunk/blender/source/blender/blenkernel/intern/writeffmpeg.c
    trunk/blender/source/blender/blenkernel/intern/writeframeserver.c
    trunk/blender/source/blender/blenlib/intern/BLI_args.c
    trunk/blender/source/blender/blenlib/intern/BLI_dynstr.c
    trunk/blender/source/blender/blenlib/intern/BLI_kdopbvh.c
    trunk/blender/source/blender/blenlib/intern/BLI_kdtree.c
    trunk/blender/source/blender/blenlib/intern/BLI_linklist.c
    trunk/blender/source/blender/blenlib/intern/bpath.c
    trunk/blender/source/blender/blenlib/intern/callbacks.c
    trunk/blender/source/blender/blenlib/intern/dynlib.c
    trunk/blender/source/blender/blenlib/intern/fileops.c
    trunk/blender/source/blender/blenlib/intern/freetypefont.c
    trunk/blender/source/blender/blenlib/intern/graph.c
    trunk/blender/source/blender/blenlib/intern/gsqueue.c
    trunk/blender/source/blender/blenlib/intern/jitter.c
    trunk/blender/source/blender/blenlib/intern/listbase.c
    trunk/blender/source/blender/blenlib/intern/math_base.c
    trunk/blender/source/blender/blenlib/intern/math_base_inline.c
    trunk/blender/source/blender/blenlib/intern/math_color.c
    trunk/blender/source/blender/blenlib/intern/math_color_inline.c
    trunk/blender/source/blender/blenlib/intern/math_geom.c
    trunk/blender/source/blender/blenlib/intern/math_geom_inline.c
    trunk/blender/source/blender/blenlib/intern/math_matrix.c
    trunk/blender/source/blender/blenlib/intern/math_rotation.c
    trunk/blender/source/blender/blenlib/intern/math_vector.c
    trunk/blender/source/blender/blenlib/intern/math_vector_inline.c
    trunk/blender/source/blender/blenlib/intern/noise.c
    trunk/blender/source/blender/blenlib/intern/path_util.c
    trunk/blender/source/blender/blenlib/intern/pbvh.c
    trunk/blender/source/blender/blenlib/intern/rand.c
    trunk/blender/source/blender/blenlib/intern/scanfill.c
    trunk/blender/source/blender/blenlib/intern/storage.c
    trunk/blender/source/blender/blenlib/intern/string.c
    trunk/blender/source/blender/blenlib/intern/string_utf8.c
    trunk/blender/source/blender/blenlib/intern/threads.c
    trunk/blender/source/blender/blenlib/intern/time.c
    trunk/blender/source/blender/blenlib/intern/winstuff.c
    trunk/blender/source/blender/blenloader/intern/readblenentry.c
    trunk/blender/source/blender/blenloader/intern/readfile.c
    trunk/blender/source/blender/blenloader/intern/runtime.c
    trunk/blender/source/blender/blenloader/intern/undofile.c
    trunk/blender/source/blender/blenloader/intern/writefile.c

Modified: trunk/blender/source/blender/blenkernel/intern/DerivedMesh.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/DerivedMesh.c	2012-03-24 04:39:53 UTC (rev 45124)
+++ trunk/blender/source/blender/blenkernel/intern/DerivedMesh.c	2012-03-24 06:18:31 UTC (rev 45125)
@@ -173,7 +173,7 @@
 	MVert *tmp = MEM_callocN(sizeof(*tmp) * dm->getNumVerts(dm),
 							 "dm_dupVertArray tmp");
 
-	if(tmp) dm->copyVertArray(dm, tmp);
+	if (tmp) dm->copyVertArray(dm, tmp);
 
 	return tmp;
 }
@@ -183,7 +183,7 @@
 	MEdge *tmp = MEM_callocN(sizeof(*tmp) * dm->getNumEdges(dm),
 							 "dm_dupEdgeArray tmp");
 
-	if(tmp) dm->copyEdgeArray(dm, tmp);
+	if (tmp) dm->copyEdgeArray(dm, tmp);
 
 	return tmp;
 }
@@ -193,7 +193,7 @@
 	MFace *tmp = MEM_callocN(sizeof(*tmp) * dm->getNumTessFaces(dm),
 							 "dm_dupFaceArray tmp");
 
-	if(tmp) dm->copyTessFaceArray(dm, tmp);
+	if (tmp) dm->copyTessFaceArray(dm, tmp);
 
 	return tmp;
 }
@@ -203,7 +203,7 @@
 	MLoop *tmp = MEM_callocN(sizeof(*tmp) * dm->getNumLoops(dm),
 							 "dm_dupLoopArray tmp");
 
-	if(tmp) dm->copyLoopArray(dm, tmp);
+	if (tmp) dm->copyLoopArray(dm, tmp);
 
 	return tmp;
 }
@@ -213,7 +213,7 @@
 	MPoly *tmp = MEM_callocN(sizeof(*tmp) * dm->getNumPolys(dm),
 							 "dm_dupPolyArray tmp");
 
-	if(tmp) dm->copyPolyArray(dm, tmp);
+	if (tmp) dm->copyPolyArray(dm, tmp);
 
 	return tmp;
 }
@@ -501,11 +501,11 @@
 	
 	/* not all DerivedMeshes store their verts/edges/faces in CustomData, so
 	 * we set them here in case they are missing */
-	if(!CustomData_has_layer(&tmp.vdata, CD_MVERT))
+	if (!CustomData_has_layer(&tmp.vdata, CD_MVERT))
 		CustomData_add_layer(&tmp.vdata, CD_MVERT, CD_ASSIGN, dm->dupVertArray(dm), totvert);
-	if(!CustomData_has_layer(&tmp.edata, CD_MEDGE))
+	if (!CustomData_has_layer(&tmp.edata, CD_MEDGE))
 		CustomData_add_layer(&tmp.edata, CD_MEDGE, CD_ASSIGN, dm->dupEdgeArray(dm), totedge);
-	if(!CustomData_has_layer(&tmp.pdata, CD_MPOLY)) {
+	if (!CustomData_has_layer(&tmp.pdata, CD_MPOLY)) {
 		tmp.mloop = dm->dupLoopArray(dm);
 		tmp.mpoly = dm->dupPolyArray(dm);
 
@@ -515,7 +515,7 @@
 
 	/* object had got displacement layer, should copy this layer to save sculpted data */
 	/* NOTE: maybe some other layers should be copied? nazgul */
-	if(CustomData_has_layer(&me->ldata, CD_MDISPS)) {
+	if (CustomData_has_layer(&me->ldata, CD_MDISPS)) {
 		if (totloop == me->totloop) {
 			MDisps *mdisps = CustomData_get_layer(&me->ldata, CD_MDISPS);
 			CustomData_add_layer(&tmp.ldata, CD_MDISPS, CD_DUPLICATE, mdisps, totloop);
@@ -536,9 +536,9 @@
 	/* ok, this should now use new CD shapekey data,
 	 * which shouuld be fed through the modifier
 	 * stack*/
-	if(tmp.totvert != me->totvert && !did_shapekeys && me->key) {
+	if (tmp.totvert != me->totvert && !did_shapekeys && me->key) {
 		printf("YEEK! this should be recoded! Shape key loss!!!\n");
-		if(tmp.key) tmp.key->id.us--;
+		if (tmp.key) tmp.key->id.us--;
 		tmp.key = NULL;
 	}
 
@@ -551,16 +551,16 @@
 	float *fp;
 	MVert *mvert;
 	
-	if(totvert==0 || me->totvert==0 || me->totvert!=totvert) return;
+	if (totvert==0 || me->totvert==0 || me->totvert!=totvert) return;
 	
-	if(kb->data) MEM_freeN(kb->data);
+	if (kb->data) MEM_freeN(kb->data);
 	kb->data= MEM_callocN(me->key->elemsize*me->totvert, "kb->data");
 	kb->totelem= totvert;
 	
 	fp= kb->data;
 	mvert=dm->getVertDataArray(dm, CD_MVERT);
 	
-	for(a=0; a<kb->totelem; a++, fp+=3, mvert++) {
+	for (a=0; a<kb->totelem; a++, fp+=3, mvert++) {
 		copy_v3_v3(fp, mvert->co);
 	}
 }
@@ -614,7 +614,7 @@
 
 void *DM_get_vert_data_layer(DerivedMesh *dm, int type)
 {
-	if(type == CD_MVERT)
+	if (type == CD_MVERT)
 		return dm->getVertArray(dm);
 
 	return CustomData_get_layer(&dm->vertData, type);
@@ -622,7 +622,7 @@
 
 void *DM_get_edge_data_layer(DerivedMesh *dm, int type)
 {
-	if(type == CD_MEDGE)
+	if (type == CD_MEDGE)
 		return dm->getEdgeArray(dm);
 
 	return CustomData_get_layer(&dm->edgeData, type);
@@ -773,7 +773,7 @@
 {
 	DerivedMesh *dm = CDDM_from_mesh(me, ob);
 	
-	if(!dm)
+	if (!dm)
 		return NULL;
 	
 	if (vertCos)
@@ -814,7 +814,8 @@
 			add_shapekey_layers(dm, me, ob);
 		
 		MEM_freeN(deformedVerts);
-	} else {
+	}
+	else {
 		DerivedMesh *tdm = mesh_create_derived(me, ob, NULL);
 
 		if (build_shapekey_layers)
@@ -822,7 +823,7 @@
 		
 		dm = mti->applyModifier(md, ob, tdm, 0, 0);
 
-		if(tdm != dm) tdm->release(tdm);
+		if (tdm != dm) tdm->release(tdm);
 	}
 
 	return dm;
@@ -855,23 +856,23 @@
 {
 	*free= 0;
 
-	if(layer == CD_ORCO) {
+	if (layer == CD_ORCO) {
 		/* get original coordinates */
 		*free= 1;
 
-		if(em)
+		if (em)
 			return (float(*)[3])get_editbmesh_orco_verts(em);
 		else
 			return (float(*)[3])get_mesh_orco_verts(ob);
 	}
-	else if(layer == CD_CLOTH_ORCO) {
+	else if (layer == CD_CLOTH_ORCO) {
 		/* apply shape key for cloth, this should really be solved
 		 * by a more flexible customdata system, but not simple */
-		if(!em) {
+		if (!em) {
 			ClothModifierData *clmd = (ClothModifierData *)modifiers_findByType(ob, eModifierType_Cloth);
 			KeyBlock *kb= key_get_keyblock(ob_get_key(ob), clmd->sim_parms->shapekey_rest);
 
-			if(kb->data)
+			if (kb->data)
 				return kb->data;
 		}
 
@@ -887,14 +888,14 @@
 	float (*orco)[3];
 	int free;
 
-	if(em) dm= CDDM_from_BMEditMesh(em, me, FALSE, FALSE);
+	if (em) dm= CDDM_from_BMEditMesh(em, me, FALSE, FALSE);
 	else dm= CDDM_from_mesh(me, ob);
 
 	orco= get_orco_coords_dm(ob, em, layer, &free);
 
-	if(orco) {
+	if (orco) {
 		CDDM_apply_vert_coords(dm, orco);
-		if(free) MEM_freeN(orco);
+		if (free) MEM_freeN(orco);
 	}
 
 	CDDM_calc_normals(dm);
@@ -910,11 +911,11 @@
 
 	totvert= dm->getNumVerts(dm);
 
-	if(orcodm) {
+	if (orcodm) {
 		orco= MEM_callocN(sizeof(float)*3*totvert, "dm orco");
 		free= 1;
 
-		if(orcodm->getNumVerts(orcodm) == totvert)
+		if (orcodm->getNumVerts(orcodm) == totvert)
 			orcodm->getVertCos(orcodm, orco);
 		else
 			dm->getVertCos(dm, orco);
@@ -922,17 +923,17 @@
 	else
 		orco= get_orco_coords_dm(ob, em, layer, &free);
 
-	if(orco) {
-		if(layer == CD_ORCO)
+	if (orco) {
+		if (layer == CD_ORCO)
 			transform_mesh_orco_verts(ob->data, orco, totvert, 0);
 
-		if(!(layerorco = DM_get_vert_data_layer(dm, layer))) {
+		if (!(layerorco = DM_get_vert_data_layer(dm, layer))) {
 			DM_add_vert_layer(dm, layer, CD_CALLOC, NULL);
 			layerorco = DM_get_vert_data_layer(dm, layer);
 		}
 
 		memcpy(layerorco, orco, sizeof(float)*3*totvert);
-		if(free) MEM_freeN(orco);
+		if (free) MEM_freeN(orco);
 	}
 }
 
@@ -987,7 +988,7 @@
 {
 	float colf[4];
 
-	if(coba) {
+	if (coba) {
 		do_colorband(coba, input, colf);
 	}
 	else {
@@ -1149,7 +1150,7 @@
 # else
 #if 0
 	/* XXX We have to create a CD_PREVIEW_MCOL, else it might sigsev (after a SubSurf mod, eg)... */
-	if(!dm->getTessFaceDataArray(dm, CD_PREVIEW_MCOL))
+	if (!dm->getTessFaceDataArray(dm, CD_PREVIEW_MCOL))
 		CustomData_add_layer(&dm->faceData, CD_PREVIEW_MCOL, CD_CALLOC, NULL, numFaces);
 #endif
 
@@ -1161,7 +1162,7 @@
 			float *w = weights;
 			/* If indices is not NULL, it means we do not have weights for all vertices,
 			 * so we must create them (and set them to zero)... */
-			if(indices) {
+			if (indices) {
 				w = MEM_callocN(sizeof(float)*numVerts, "Temp weight array DM_update_weight_mcol");
 				i = num;
 				while(i--)
@@ -1171,7 +1172,7 @@
 			/* Convert float weights to colors. */
 			wtcol_v = calc_colors_from_weights_array(numVerts, w);
 
-			if(indices)
+			if (indices)
 				MEM_freeN(w);
 		}
 
@@ -1277,7 +1278,8 @@
 			for (j=0; j<dm->numVertData; j++, kbcos++, mvert++) {
 				copy_v3_v3(*kbcos, mvert->co);
 			}
-		} else {
+		}
+		else {
 			for (j=0; j<kb->totelem; j++, cos++, kbcos++) {
 				copy_v3_v3(*kbcos, *cos);
 			}
@@ -1374,16 +1376,16 @@
 	/* XXX Same as above... For now, only weights preview in WPaint mode. */
 	const int do_mod_wmcol = do_init_wmcol;
 
-	if(mmd && !mmd->sculptlvl)
+	if (mmd && !mmd->sculptlvl)
 		has_multires = 0;
 
-	if(!skipVirtualArmature) {
+	if (!skipVirtualArmature) {
 		firstmd = modifiers_getVirtualModifierList(ob);
 	}
 	else {
 		/* game engine exception */
 		firstmd = ob->modifiers.first;
-		if(firstmd && firstmd->type == eModifierType_Armature)
+		if (firstmd && firstmd->type == eModifierType_Armature)
 			firstmd = firstmd->next;
 	}
 
@@ -1391,13 +1393,13 @@
 
 	modifiers_clearErrors(ob);
 
-	if(useRenderParams) required_mode = eModifierMode_Render;
+	if (useRenderParams) required_mode = eModifierMode_Render;
 	else required_mode = eModifierMode_Realtime;
 
 	datamasks = modifiers_calcDataMasks(scene, ob, md, dataMask, required_mode);
 	curr = datamasks;
 
-	if(do_mod_wmcol || do_mod_mcol) {
+	if (do_mod_wmcol || do_mod_mcol) {
 		/* Find the last active modifier generating a preview, or NULL if none. */
 		/* XXX Currently, DPaint modifier just ignores this.
 		 *     Needs a stupid hack...
@@ -1405,33 +1407,34 @@
 		previewmd = modifiers_getLastPreview(scene, md, required_mode);
 	}
 
-	if(deform_r) *deform_r = NULL;
+	if (deform_r) *deform_r = NULL;
 	*final_r = NULL;
 
-	if(useDeform) {
-		if(inputVertexCos)
+	if (useDeform) {
+		if (inputVertexCos)
 			deformedVerts = inputVertexCos;
 		
 		/* Apply all leading deforming modifiers */
-		for(;md; md = md->next, curr = curr->next) {
+		for (;md; md = md->next, curr = curr->next) {
 			ModifierTypeInfo *mti = modifierType_getInfo(md->type);
 
 			md->scene= scene;
 			
-			if(!modifier_isEnabled(scene, md, required_mode)) continue;
-			if(useDeform < 0 && mti->dependsOnTime && mti->dependsOnTime(md)) continue;
+			if (!modifier_isEnabled(scene, md, required_mode)) continue;
+			if (useDeform < 0 && mti->dependsOnTime && mti->dependsOnTime(md)) continue;
 
-			if(mti->type == eModifierTypeType_OnlyDeform) {
-				if(!deformedVerts)
+			if (mti->type == eModifierTypeType_OnlyDeform) {
+				if (!deformedVerts)
 					deformedVerts = mesh_getVertexCos(me, &numVerts);
 
 				mti->deformVerts(md, ob, NULL, deformedVerts, numVerts, useRenderParams, useDeform);
-			} else {
+			}
+			else {
 				break;
 			}
 			
 			/* grab modifiers until index i */
-			if((index >= 0) && (modifiers_indexInObject(ob, md) >= index))
+			if ((index >= 0) && (modifiers_indexInObject(ob, md) >= index))
 				break;
 		}
 
@@ -1445,14 +1448,15 @@
 			if (build_shapekey_layers)
 				add_shapekey_layers(dm, me, ob);
 			
-			if(deformedVerts) {
+			if (deformedVerts) {
 				CDDM_apply_vert_coords(*deform_r, deformedVerts);
 				CDDM_calc_normals(*deform_r);
 			}
 		}
-	} else {
+	}
+	else {
 		/* default behavior for meshes */
-		if(inputVertexCos)

@@ Diff output truncated at 10240 characters. @@



More information about the Bf-blender-cvs mailing list