[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [40180] branches/bmesh/blender/source/ blender/bmesh/operators/edgesplitop.c: picky formatting edits - no functional changes.

Campbell Barton ideasman42 at gmail.com
Tue Sep 13 11:43:58 CEST 2011


Revision: 40180
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=40180
Author:   campbellbarton
Date:     2011-09-13 09:43:57 +0000 (Tue, 13 Sep 2011)
Log Message:
-----------
picky formatting edits - no functional changes.

Modified Paths:
--------------
    branches/bmesh/blender/source/blender/bmesh/operators/edgesplitop.c

Modified: branches/bmesh/blender/source/blender/bmesh/operators/edgesplitop.c
===================================================================
--- branches/bmesh/blender/source/blender/bmesh/operators/edgesplitop.c	2011-09-13 09:07:20 UTC (rev 40179)
+++ branches/bmesh/blender/source/blender/bmesh/operators/edgesplitop.c	2011-09-13 09:43:57 UTC (rev 40180)
@@ -83,16 +83,18 @@
 	for (i=0; i<f->len-1; i++) {
 		edges[i] = BM_Make_Edge(bm, verts[i], verts[i+1], NULL, 1);
 
-		if (!edges[i])
+		if (!edges[i]) {
 			return NULL;
+		}
 	}
-	
+
 	edges[i] = BM_Make_Edge(bm, lastv1, lastv2, NULL, 1);
 
 	f2 = BM_Make_Ngon(bm, v1, v2, edges, f->len, 0);
-	if (!f2)
+	if (!f2) {
 		return NULL;
-	
+	}
+
 	BM_Copy_Attributes(bm, bm, f, f2);
 
 	l = BMIter_New(&liter1, bm, BM_LOOPS_OF_FACE, f);
@@ -105,11 +107,16 @@
 			BM_SetIndex(l2->e, BM_GetIndex(l->e));
 			et = etags + BM_GetIndex(l->e);
 			
-			if (!et->newe1) et->newe1 = l2->e;
-			else et->newe2 = l2->e;
+			if (!et->newe1) {
+				et->newe1 = l2->e;
+			}
+			else {
+				et->newe2 = l2->e;
+			}
 
-			if (BMO_TestFlag(bm, l->e, EDGE_SEAM))
+			if (BMO_TestFlag(bm, l->e, EDGE_SEAM)) {
 				BMO_SetFlag(bm, l2->e, EDGE_SEAM);
+			}
 
 			BM_Copy_Attributes(bm, bm, l->e, l2->e);
 		}
@@ -163,7 +170,8 @@
 						BMO_SetFlag(bm, et->newe2, EDGE_RET2);
 						BMO_ClearFlag(bm, et->newe2, EDGE_SEAM);
 					}
-				} else {
+				}
+				else {
 					if (et->newe1) {
 						BMO_SetFlag(bm, et->newe1, EDGE_RET2);
 						BMO_ClearFlag(bm, et->newe1, EDGE_SEAM);
@@ -177,17 +185,17 @@
 				startl = l;
 				do {
 					l = BM_OtherFaceLoop(l->e, l->f, v);
-					if (BM_Edge_FaceCount(l->e) != 2)
+					if (BM_Edge_FaceCount(l->e) != 2) {
 						break;
+					}
 					l = l->radial_next;
 				} while (l != startl && !BMO_TestFlag(bm, l->e, EDGE_SEAM));
 				
-				if (l == startl || !BMO_TestFlag(bm, l->e, EDGE_SEAM))
+				if (l == startl || !BMO_TestFlag(bm, l->e, EDGE_SEAM)) {
 					break;
+				}
 
-				if (l->v == v) {
-					v = l->next->v;
-				} else v = l->v;
+				v = (l->v == v) ? l->next->v : l->v;
 			}
 		}
 	}
@@ -212,14 +220,18 @@
 	BMO_ITER(e, &siter, bm, op, "edges", BM_EDGE) {
 		for (i=0; i<2; i++) {
 			BM_ITER(e2, &iter, bm, BM_EDGES_OF_VERT, i ? e->v2 : e->v1) {
-				if (e != e2 && BMO_TestFlag(bm, e2, EDGE_SEAM))
+				if (e != e2 && BMO_TestFlag(bm, e2, EDGE_SEAM)) {
 					break;
+				}
 			}
-			if (e2)
+			if (e2) {
 				break;
+			}
 		}
-		if (!e2)
+
+		if (!e2) {
 			BMO_ClearFlag(bm, e, EDGE_SEAM);
+		}
 	}
 
 	etags = MEM_callocN(sizeof(EdgeTag)*bm->totedge, "EdgeTag");
@@ -241,9 +253,11 @@
 #define SETETV(et, v, l, vs) l->e->v1 == v ? (et->newv1 = vs) : (et->newv2 = vs)
 
 	BM_ITER(f, &iter, bm, BM_FACES_OF_MESH, NULL) {
-		if (BMO_TestFlag(bm, f, FACE_NEW))
+
+		if (BMO_TestFlag(bm, f, FACE_NEW)) {
 			continue;
-		
+		}
+
 		BLI_array_empty(verts);
 		BLI_array_growitems(verts, f->len);
 		memset(verts, 0, sizeof(BMVert*)*f->len);
@@ -253,9 +267,12 @@
 			if (!BMO_TestFlag(bm, l->e, EDGE_SEAM)) {
 				if (!verts[i]) {
 					et = etags + BM_GetIndex(l->e);
-					if (ETV(et, l->v, l))
+					if (ETV(et, l->v, l)) {
 						verts[i] = ETV(et, l->v, l);
-					else verts[i] = l->v;
+					}
+					else {
+						verts[i] = l->v;
+					}
 				}
 				i++;
 				continue;
@@ -274,21 +291,26 @@
 						v2 = BM_Make_Vert(bm, v->co, v);
 
 						verts[j ? (i+1) % f->len : i] = v2;
-					} else v2 = verts[j ? (i+1) % f->len : i];
-				} else {
-					/*generate unique vert for non-seam edge(s)
-					  around the manifold vert fan if necassary*/
+					}
+					else {
+						v2 = verts[j ? (i+1) % f->len : i];
+					}
+				}
+				else {
+					/* generate unique vert for non-seam edge(s)
+					 * around the manifold vert fan if necassary */
 
-					/*first check that we have two seam edges
-					  somewhere within this fan*/
+					/* first check that we have two seam edges
+					 * somewhere within this fan*/
 					l3 = l2;
 					do {
 						if (BM_Edge_FaceCount(l3->e) != 2) {
-							/*if we hit a boundary edge, tag
-							  l3 as null so we know to disconnect
-							  it*/
-							if (BM_Edge_FaceCount(l3->e) == 1)
+							/* if we hit a boundary edge, tag
+							 * l3 as null so we know to disconnect
+							 * it */
+							if (BM_Edge_FaceCount(l3->e) == 1) {
 								l3 = NULL;
+							}
 							break;
 						}
 
@@ -304,18 +326,25 @@
 							l3 = l2;
 							do {
 								SETETV(et, v, l3, v2);
-								if (BM_Edge_FaceCount(l3->e) != 2)
+								if (BM_Edge_FaceCount(l3->e) != 2) {
 									break;
+								}
 
 								l3 = l3->radial_next;
 								l3 = BM_OtherFaceLoop(l3->e, l3->f, v);
 								
 								et = etags + BM_GetIndex(l3->e);
 							} while (l3 != l2 && !BMO_TestFlag(bm, l3->e, EDGE_SEAM));
-						} else v2 = ETV(et, v, l2);
+						}
+						else {
+							v2 = ETV(et, v, l2);
+						}
 
 						verts[j ? (i+1) % f->len : i] = v2;
-					} else verts[j ? (i+1) % f->len : i] = v;
+					}
+					else {
+						verts[j ? (i+1) % f->len : i] = v;
+					}
 				}
 			}
 
@@ -323,8 +352,9 @@
 		}
 
 		f2 = remake_face(bm, etags, f, verts);
-		if (!f2)
+		if (!f2) {
 			continue;
+		}
 
 		BMO_SetFlag(bm, f, FACE_DEL);
 		BMO_SetFlag(bm, f2, FACE_NEW);
@@ -336,8 +366,9 @@
 	  is set in remake_face*/
 	BM_ITER(e, &iter, bm, BM_EDGES_OF_MESH, NULL) {
 		if (BMO_TestFlag(bm, e, EDGE_MARK)) {
-			if (!e->l)
+			if (!e->l) {
 				BMO_SetFlag(bm, e, EDGE_DEL);
+			}
 		}
 	}
 




More information about the Bf-blender-cvs mailing list