[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [44729] trunk/blender/source/blender: code cleanup: duplicate checks and double assignments.

Campbell Barton ideasman42 at gmail.com
Thu Mar 8 05:38:44 CET 2012


Revision: 44729
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=44729
Author:   campbellbarton
Date:     2012-03-08 04:38:35 +0000 (Thu, 08 Mar 2012)
Log Message:
-----------
code cleanup: duplicate checks and double assignments.

Modified Paths:
--------------
    trunk/blender/source/blender/blenkernel/intern/cdderivedmesh.c
    trunk/blender/source/blender/blenkernel/intern/mesh.c
    trunk/blender/source/blender/bmesh/operators/bmo_primitive.c
    trunk/blender/source/blender/editors/mesh/editface.c
    trunk/blender/source/blender/editors/mesh/loopcut.c
    trunk/blender/source/blender/gpu/intern/gpu_extensions.c

Modified: trunk/blender/source/blender/blenkernel/intern/cdderivedmesh.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/cdderivedmesh.c	2012-03-08 04:12:11 UTC (rev 44728)
+++ trunk/blender/source/blender/blenkernel/intern/cdderivedmesh.c	2012-03-08 04:38:35 UTC (rev 44729)
@@ -2224,7 +2224,6 @@
 	
 	/*find-replace merged vertices with target vertices*/	
 	ml = cddm->mloop;
-	c = 0;
 	for (i=0; i<totloop; i++, ml++) {
 		if (vtargetmap[ml->v] != -1) {
 			ml->v = vtargetmap[ml->v];

Modified: trunk/blender/source/blender/blenkernel/intern/mesh.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/mesh.c	2012-03-08 04:12:11 UTC (rev 44728)
+++ trunk/blender/source/blender/blenkernel/intern/mesh.c	2012-03-08 04:38:35 UTC (rev 44729)
@@ -567,9 +567,7 @@
 				for (a=0; a<me->totpoly; a++, txface++) {
 					/* special case: ima always local immediately */
 					if (txface->tpage) {
-						if (txface->tpage) {
-							id_lib_extern((ID *)txface->tpage);
-						}
+						id_lib_extern((ID *)txface->tpage);
 					}
 				}
 			}
@@ -582,9 +580,7 @@
 				for (a=0; a<me->totface; a++, tface++) {
 					/* special case: ima always local immediately */
 					if (tface->tpage) {
-						if (tface->tpage) {
-							id_lib_extern((ID *)tface->tpage);
-						}
+						id_lib_extern((ID *)tface->tpage);
 					}
 				}
 			}

Modified: trunk/blender/source/blender/bmesh/operators/bmo_primitive.c
===================================================================
--- trunk/blender/source/blender/bmesh/operators/bmo_primitive.c	2012-03-08 04:12:11 UTC (rev 44728)
+++ trunk/blender/source/blender/bmesh/operators/bmo_primitive.c	2012-03-08 04:38:35 UTC (rev 44729)
@@ -299,7 +299,7 @@
 	BMO_slot_mat4_get(op, "mat", mat);
 
 	phid = 2.0f * (float)M_PI / tot;
-	phi = 0.25f * (float)M_PI;
+	/* phi = 0.25f * (float)M_PI; */ /* UNUSED */
 
 	/* one segment first */
 	phi = 0;

Modified: trunk/blender/source/blender/editors/mesh/editface.c
===================================================================
--- trunk/blender/source/blender/editors/mesh/editface.c	2012-03-08 04:12:11 UTC (rev 44728)
+++ trunk/blender/source/blender/editors/mesh/editface.c	2012-03-08 04:38:35 UTC (rev 44729)
@@ -576,10 +576,9 @@
 
 	selar= MEM_callocN(me->totpoly+1, "selar");
 
-	if (extend == 0 && select)
+	if (extend == 0 && select) {
 		paintface_deselect_all_visible(vc->obact, SEL_DESELECT, FALSE);
 
-	if (extend == 0 && select) {
 		mface= me->mpoly;
 		for(a=1; a<=me->totpoly; a++, mface++) {
 			if((mface->flag & ME_HIDE) == 0)

Modified: trunk/blender/source/blender/editors/mesh/loopcut.c
===================================================================
--- trunk/blender/source/blender/editors/mesh/loopcut.c	2012-03-08 04:12:11 UTC (rev 44728)
+++ trunk/blender/source/blender/editors/mesh/loopcut.c	2012-03-08 04:38:35 UTC (rev 44729)
@@ -289,9 +289,9 @@
 {
 	if (lcd->eed) {
 		edgering_sel(lcd, cuts, 0);
-	} else if(lcd->edges) {
-		if (lcd->edges)
-			MEM_freeN(lcd->edges);
+	}
+	else if (lcd->edges) {
+		MEM_freeN(lcd->edges);
 		lcd->edges = NULL;
 		lcd->totedge = 0;
 	}

Modified: trunk/blender/source/blender/gpu/intern/gpu_extensions.c
===================================================================
--- trunk/blender/source/blender/gpu/intern/gpu_extensions.c	2012-03-08 04:12:11 UTC (rev 44728)
+++ trunk/blender/source/blender/gpu/intern/gpu_extensions.c	2012-03-08 04:38:35 UTC (rev 44729)
@@ -183,12 +183,12 @@
 	 * really needed to prevent different unexpected behaviors like with intel gme965 card (sergey) */
 	glLightModeli(GL_LIGHT_MODEL_TWO_SIDE, GL_FALSE);
 
-	GG.os = GPU_OS_UNIX;
 #ifdef _WIN32
 	GG.os = GPU_OS_WIN;
-#endif
-#ifdef __APPLE__
+#elif defined(__APPLE__)
 	GG.os = GPU_OS_MAC;
+#else
+	GG.os = GPU_OS_UNIX;
 #endif
 }
 




More information about the Bf-blender-cvs mailing list