[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [58219] trunk/blender/source/blender: fix for bad NULL check in bmo_connect_pair, also remove duplicate checks in if statements and redundant initialization vars .

[Campbell Barton]

Revision: 58219
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=58219
Author:   campbellbarton
Date:     2013-07-13 16:25:47 +0000 (Sat, 13 Jul 2013)
Log Message:
-----------
fix for bad NULL check in bmo_connect_pair, also remove duplicate checks in if statements and redundant initialization vars.

Modified Paths:
--------------
    trunk/blender/source/blender/blenfont/intern/blf_font.c
    trunk/blender/source/blender/blenkernel/intern/brush.c
    trunk/blender/source/blender/blenkernel/intern/constraint.c
    trunk/blender/source/blender/blenkernel/intern/tracking.c
    trunk/blender/source/blender/bmesh/operators/bmo_connect_pair.c
    trunk/blender/source/blender/editors/mesh/editmesh_utils.c
    trunk/blender/source/blender/editors/transform/transform_manipulator.c
    trunk/blender/source/blender/render/intern/source/convertblender.c
    trunk/blender/source/blender/render/intern/source/render_texture.c

Modified: trunk/blender/source/blender/blenfont/intern/blf_font.c
===================================================================
--- trunk/blender/source/blender/blenfont/intern/blf_font.c	2013-07-13 14:44:04 UTC (rev 58218)
+++ trunk/blender/source/blender/blenfont/intern/blf_font.c	2013-07-13 16:25:47 UTC (rev 58219)
@@ -87,7 +87,7 @@
 	err = FT_Set_Char_Size(font->face, 0, (FT_F26Dot6)(size * 64), dpi, dpi);
 	if (err) {
 		/* FIXME: here we can go through the fixed size and choice a close one */
-		printf("The current font don't support the size, %d and dpi, %d\n", size, dpi);
+		printf("The current font don't support the size, %u and dpi, %u\n", size, dpi);
 		return;
 	}
 

Modified: trunk/blender/source/blender/blenkernel/intern/brush.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/brush.c	2013-07-13 14:44:04 UTC (rev 58218)
+++ trunk/blender/source/blender/blenkernel/intern/brush.c	2013-07-13 16:25:47 UTC (rev 58219)
@@ -539,7 +539,7 @@
 	else if (mtex->brush_map_mode == MTEX_MAP_MODE_STENCIL) {
 		float rotation = -mtex->rot;
 		float point_2d[2] = {point[0], point[1]};
-		float x = 0.0f, y = 0.0f; /* Quite warnings */
+		float x, y;
 		float co[3];
 
 		x = point_2d[0] - br->stencil_pos[0];
@@ -658,7 +658,7 @@
 	if (mtex->brush_map_mode == MTEX_MAP_MODE_STENCIL) {
 		float rotation = -mtex->rot;
 		float point_2d[2] = {point[0], point[1]};
-		float x = 0.0f, y = 0.0f; /* Quite warnings */
+		float x, y;
 		float co[3];
 
 		x = point_2d[0] - br->mask_stencil_pos[0];

Modified: trunk/blender/source/blender/blenkernel/intern/constraint.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/constraint.c	2013-07-13 14:44:04 UTC (rev 58218)
+++ trunk/blender/source/blender/blenkernel/intern/constraint.c	2013-07-13 16:25:47 UTC (rev 58219)
@@ -2519,7 +2519,7 @@
 	
 	/* only evaluate if there is a target */
 	if (VALID_CONS_TARGET(ct)) {
-		float dvec[3], dist = 0.0f, sfac = 1.0f;
+		float dvec[3], dist, sfac = 1.0f;
 		short clamp_surf = 0;
 		
 		/* calculate our current distance from the target */

Modified: trunk/blender/source/blender/blenkernel/intern/tracking.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/tracking.c	2013-07-13 14:44:04 UTC (rev 58218)
+++ trunk/blender/source/blender/blenkernel/intern/tracking.c	2013-07-13 16:25:47 UTC (rev 58219)
@@ -3455,7 +3455,7 @@
 		prev = i;
 	}
 
-	return count % 2 ? true : false;
+	return (count % 2) ? true : false;
 }
 
 /* Check whether point is inside any stroke of grease pencil layer. */

Modified: trunk/blender/source/blender/bmesh/operators/bmo_connect_pair.c
===================================================================
--- trunk/blender/source/blender/bmesh/operators/bmo_connect_pair.c	2013-07-13 14:44:04 UTC (rev 58218)
+++ trunk/blender/source/blender/bmesh/operators/bmo_connect_pair.c	2013-07-13 16:25:47 UTC (rev 58219)
@@ -381,7 +381,7 @@
 	pc.v_b = ((BMVert **)op_verts_slot->data.p)[1];
 
 	/* fail! */
-	if (!(pc.v_a && pc.v_a)) {
+	if (!(pc.v_a && pc.v_b)) {
 		return;
 	}
 

Modified: trunk/blender/source/blender/editors/mesh/editmesh_utils.c
===================================================================
--- trunk/blender/source/blender/editors/mesh/editmesh_utils.c	2013-07-13 14:44:04 UTC (rev 58218)
+++ trunk/blender/source/blender/editors/mesh/editmesh_utils.c	2013-07-13 16:25:47 UTC (rev 58219)
@@ -174,7 +174,6 @@
 DerivedMesh *EDBM_mesh_deform_dm_get(BMEditMesh *em)
 {
 	return ((em->derivedFinal != NULL) &&
-	        (em->derivedFinal) &&
 	        (em->derivedFinal->type == DM_TYPE_EDITBMESH) &&
 	        (em->derivedFinal->deformedOnly != false)) ? em->derivedFinal : NULL;
 }

Modified: trunk/blender/source/blender/editors/transform/transform_manipulator.c
===================================================================
--- trunk/blender/source/blender/editors/transform/transform_manipulator.c	2013-07-13 14:44:04 UTC (rev 58218)
+++ trunk/blender/source/blender/editors/transform/transform_manipulator.c	2013-07-13 16:25:47 UTC (rev 58219)
@@ -1682,7 +1682,7 @@
 	rctf rect;
 	GLuint buffer[64];      // max 4 items per select, so large enuf
 	short hits;
-	extern void setwinmatrixview3d(ARegion *ar, View3D *v3d, rctf *rect); // XXX check a bit later on this... (ton)
+	extern void setwinmatrixview3d(ARegion *, View3D *, rctf *); // XXX check a bit later on this... (ton)
 
 	/* when looking through a selected camera, the manipulator can be at the
 	 * exact same position as the view, skip so we don't break selection */

Modified: trunk/blender/source/blender/render/intern/source/convertblender.c
===================================================================
--- trunk/blender/source/blender/render/intern/source/convertblender.c	2013-07-13 14:44:04 UTC (rev 58218)
+++ trunk/blender/source/blender/render/intern/source/convertblender.c	2013-07-13 16:25:47 UTC (rev 58219)
@@ -3478,9 +3478,14 @@
 				ma= give_render_material(re, ob, a1+1);
 				
 				/* test for 100% transparent */
-				ok= 1;
-				if (ma->alpha==0.0f && ma->spectra==0.0f && ma->spectra==0.0f && ma->filter==0.0f && (ma->mode & MA_TRANSP) && (ma->mode & (MA_RAYTRANSP | MA_RAYMIRROR))==0 ) {
-					ok= 0;
+				ok = 1;
+				if ((ma->alpha == 0.0f) &&
+				    (ma->spectra == 0.0f) &&
+				    (ma->filter == 0.0f) &&
+				    (ma->mode & MA_TRANSP) &&
+				    (ma->mode & (MA_RAYTRANSP | MA_RAYMIRROR)) == 0)
+				{
+					ok = 0;
 					/* texture on transparency? */
 					for (a=0; a<MAX_MTEX; a++) {
 						if (ma->mtex[a] && ma->mtex[a]->tex) {

Modified: trunk/blender/source/blender/render/intern/source/render_texture.c
===================================================================
--- trunk/blender/source/blender/render/intern/source/render_texture.c	2013-07-13 14:44:04 UTC (rev 58218)
+++ trunk/blender/source/blender/render/intern/source/render_texture.c	2013-07-13 16:25:47 UTC (rev 58219)
@@ -409,7 +409,7 @@
 
 static int magic(Tex *tex, const float texvec[3], TexResult *texres)
 {
-	float x, y, z, turb=1.0;
+	float x, y, z, turb;
 	int n;
 
 	n= tex->noisedepth;