[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [46910] trunk/blender/source/blender: code cleanup: double promotion warnings

Campbell Barton ideasman42 at gmail.com
Tue May 22 23:17:00 CEST 2012


Revision: 46910
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=46910
Author:   campbellbarton
Date:     2012-05-22 21:16:59 +0000 (Tue, 22 May 2012)
Log Message:
-----------
code cleanup: double promotion warnings

Modified Paths:
--------------
    trunk/blender/source/blender/blenkernel/intern/brush.c
    trunk/blender/source/blender/blenkernel/intern/constraint.c
    trunk/blender/source/blender/blenkernel/intern/pointcache.c
    trunk/blender/source/blender/editors/interface/interface_regions.c
    trunk/blender/source/blender/editors/mesh/editmesh_tools.c
    trunk/blender/source/blender/editors/space_node/drawnode.c
    trunk/blender/source/blender/editors/transform/transform.c
    trunk/blender/source/blender/editors/transform/transform_conversions.c
    trunk/blender/source/blender/editors/transform/transform_input.c

Modified: trunk/blender/source/blender/blenkernel/intern/brush.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/brush.c	2012-05-22 19:16:20 UTC (rev 46909)
+++ trunk/blender/source/blender/blenkernel/intern/brush.c	2012-05-22 21:16:59 UTC (rev 46910)
@@ -1329,7 +1329,7 @@
 
 	for (i = 0; i < side; ++i) {
 		for (j = 0; j < side; ++j) {
-			float magn = sqrt(pow(i - half, 2) + pow(j - half, 2));
+			float magn = sqrtf(powf(i - half, 2) + powf(j - half, 2));
 			im->rect_float[i * side + j] = BKE_brush_curve_strength_clamp(br, magn, half);
 		}
 	}

Modified: trunk/blender/source/blender/blenkernel/intern/constraint.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/constraint.c	2012-05-22 19:16:20 UTC (rev 46909)
+++ trunk/blender/source/blender/blenkernel/intern/constraint.c	2012-05-22 21:16:59 UTC (rev 46910)
@@ -3155,7 +3155,7 @@
 					/* find bounding-box range where target is located */
 					if (ownLoc[clamp_axis] < curveMin[clamp_axis]) {
 						/* bounding-box range is before */
-						offset = curveMin[clamp_axis] - ceil((curveMin[clamp_axis] - ownLoc[clamp_axis]) / len) * len;
+						offset = curveMin[clamp_axis] - ceilf((curveMin[clamp_axis] - ownLoc[clamp_axis]) / len) * len;
 
 						/* now, we calculate as per normal, except using offset instead of curveMin[clamp_axis] */
 						curvetime = (ownLoc[clamp_axis] - offset) / (len);

Modified: trunk/blender/source/blender/blenkernel/intern/pointcache.c
===================================================================
--- trunk/blender/source/blender/blenkernel/intern/pointcache.c	2012-05-22 19:16:20 UTC (rev 46909)
+++ trunk/blender/source/blender/blenkernel/intern/pointcache.c	2012-05-22 21:16:59 UTC (rev 46910)
@@ -3292,9 +3292,9 @@
 		mb = (bytes > 1024.0f * 1024.0f);
 
 		BLI_snprintf(mem_info, sizeof(mem_info), "%i frames in memory (%.1f %s)",
-			totframes,
-			bytes / (mb ? 1024.0f * 1024.0f : 1024.0f),
-			mb ? "Mb" : "kb");
+		             totframes,
+		             bytes / (mb ? 1024.0f * 1024.0f : 1024.0f),
+		             mb ? "Mb" : "kb");
 	}
 
 	if (cache->flag & PTCACHE_OUTDATED) {

Modified: trunk/blender/source/blender/editors/interface/interface_regions.c
===================================================================
--- trunk/blender/source/blender/editors/interface/interface_regions.c	2012-05-22 19:16:20 UTC (rev 46909)
+++ trunk/blender/source/blender/editors/interface/interface_regions.c	2012-05-22 21:16:59 UTC (rev 46910)
@@ -442,7 +442,7 @@
 		if (ELEM(but->type, ROW, MENU)) {
 			EnumPropertyItem *item;
 			int i, totitem, free;
-			int value = (but->type == ROW) ? but->hardmax : ui_get_but_val(but);
+			int value = (but->type == ROW) ? (int)but->hardmax : (int)ui_get_but_val(but);
 
 			RNA_property_enum_items_gettexted(C, &but->rnapoin, but->rnaprop, &item, &totitem, &free);
 

Modified: trunk/blender/source/blender/editors/mesh/editmesh_tools.c
===================================================================
--- trunk/blender/source/blender/editors/mesh/editmesh_tools.c	2012-05-22 19:16:20 UTC (rev 46909)
+++ trunk/blender/source/blender/editors/mesh/editmesh_tools.c	2012-05-22 21:16:59 UTC (rev 46910)
@@ -4541,7 +4541,7 @@
 			mdiff[1] = opdata->mcenter[1] - event->mval[1];
 
 			factor = len_v2(mdiff) / opdata->initial_length;
-			factor = MAX2(1.0 - factor, 0.0);
+			factor = MAX2(1.0f - factor, 0.0f);
 
 			RNA_float_set(op->ptr, "percent", factor);
 
@@ -4837,7 +4837,7 @@
 			else {
 				amount = opdata->old_thickness - (len_v2(mdiff)
 				                                  - opdata->initial_length) / opdata->initial_length;
-				amount = MAX2(amount, 0.0);
+				amount = MAX2(amount, 0.0f);
 
 				RNA_float_set(op->ptr, "thickness", amount);
 			}

Modified: trunk/blender/source/blender/editors/space_node/drawnode.c
===================================================================
--- trunk/blender/source/blender/editors/space_node/drawnode.c	2012-05-22 19:16:20 UTC (rev 46909)
+++ trunk/blender/source/blender/editors/space_node/drawnode.c	2012-05-22 21:16:59 UTC (rev 46910)
@@ -2281,15 +2281,15 @@
 
 void node_composit_backdrop_boxmask(SpaceNode* snode, ImBuf* backdrop, bNode* node, int x, int y)
 {
-	NodeBoxMask * boxmask = node->storage;
+	NodeBoxMask *boxmask = node->storage;
 	const float backdropWidth = backdrop->x;
 	const float backdropHeight = backdrop->y;
 	const float aspect = backdropWidth/backdropHeight;
-	const float rad = DEG2RAD(-boxmask->rotation);
-	const float cosine = cos(rad);
-	const float sine = sin(rad);
-	const float halveBoxWidth = backdropWidth*(boxmask->width/2.0f);
-	const float halveBoxHeight = backdropHeight*(boxmask->height/2.0f)*aspect;
+	const float rad = DEG2RADF(-boxmask->rotation);
+	const float cosine = cosf(rad);
+	const float sine = sinf(rad);
+	const float halveBoxWidth = backdropWidth * (boxmask->width / 2.0f);
+	const float halveBoxHeight = backdropHeight * (boxmask->height / 2.0f) * aspect;
 
 	float cx, cy, x1, x2, x3, x4;
 	float y1, y2, y3, y4;
@@ -2325,12 +2325,12 @@
 	NodeEllipseMask * ellipsemask = node->storage;
 	const float backdropWidth = backdrop->x;
 	const float backdropHeight = backdrop->y;
-	const float aspect = backdropWidth/backdropHeight;
-	const float rad = DEG2RAD(-ellipsemask->rotation);
-	const float cosine = cos(rad);
-	const float sine = sin(rad);
-	const float halveBoxWidth = backdropWidth*(ellipsemask->width/2.0f);
-	const float halveBoxHeight = backdropHeight*(ellipsemask->height/2.0f)*aspect;
+	const float aspect = backdropWidth / backdropHeight;
+	const float rad = DEG2RADF(-ellipsemask->rotation);
+	const float cosine = cosf(rad);
+	const float sine = sinf(rad);
+	const float halveBoxWidth = backdropWidth * (ellipsemask->width / 2.0f);
+	const float halveBoxHeight = backdropHeight * (ellipsemask->height / 2.0f) * aspect;
 
 	float cx, cy, x1, x2, x3, x4;
 	float y1, y2, y3, y4;

Modified: trunk/blender/source/blender/editors/transform/transform.c
===================================================================
--- trunk/blender/source/blender/editors/transform/transform.c	2012-05-22 19:16:20 UTC (rev 46909)
+++ trunk/blender/source/blender/editors/transform/transform.c	2012-05-22 21:16:59 UTC (rev 46910)
@@ -5554,7 +5554,7 @@
 		else
 #endif
 		{
-			val= floorf(val+0.5f);
+			val = floor(val + 0.5);
 		}
 		
 		/* convert frame out of nla-action time */
@@ -5640,7 +5640,7 @@
 		/* apply snapping + frame->seconds conversions */
 		if (autosnap == SACTSNAP_STEP) {
 			if (do_time)
-				val= floorf((double)val/secf + 0.5f);
+				val= floorf((double)val / secf + 0.5);
 			else
 				val= floorf(val + 0.5f);
 		}
@@ -5688,9 +5688,9 @@
 
 			if (autosnap == SACTSNAP_STEP) {
 				if (do_time)
-					deltax= (float)(floor((deltax/secf) + 0.5f) * secf);
+					deltax = (float)(floor(((double)deltax / secf) + 0.5) * secf);
 				else
-					deltax= (float)(floor(deltax + 0.5f));
+					deltax = (float)(floor(deltax + 0.5f));
 			}
 
 			val = BKE_nla_tweakedit_remap(adt, td->ival, NLATIME_CONVERT_MAP);
@@ -5702,9 +5702,9 @@
 
 			if (autosnap == SACTSNAP_STEP) {
 				if (do_time)
-					val= (float)(floor((deltax/secf) + 0.5f) * secf);
+					val = (float)(floor(((double)deltax / secf) + 0.5) * secf);
 				else
-					val= (float)(floor(val + 0.5f));
+					val = (float)(floor(val + 0.5f));
 			}
 
 			*(td->val) = td->ival + val;
@@ -5954,9 +5954,9 @@
 
 		if (autosnap == SACTSNAP_STEP) {
 			if (do_time)
-				fac= (float)(floor(fac/secf + 0.5f) * secf);
+				fac = (float)(floor((double)fac / secf + 0.5) * secf);
 			else
-				fac= (float)(floor(fac + 0.5f));
+				fac = (float)(floor(fac + 0.5f));
 		}
 
 		/* check if any need to apply nla-mapping */
@@ -5964,9 +5964,7 @@
 			startx= BKE_nla_tweakedit_remap(adt, startx, NLATIME_CONVERT_UNMAP);
 
 		/* now, calculate the new value */
-		*(td->val) = td->ival - startx;
-		*(td->val) *= fac;
-		*(td->val) += startx;
+		*(td->val) = ((td->ival - startx) * fac) + startx;
 
 		/* apply nearest snapping */
 		doAnimEdit_SnapFrame(t, td, td2d, adt, autosnap);

Modified: trunk/blender/source/blender/editors/transform/transform_conversions.c
===================================================================
--- trunk/blender/source/blender/editors/transform/transform_conversions.c	2012-05-22 19:16:20 UTC (rev 46909)
+++ trunk/blender/source/blender/editors/transform/transform_conversions.c	2012-05-22 21:16:59 UTC (rev 46910)
@@ -3722,9 +3722,9 @@
 			switch (sipo->autosnap) {
 				case SACTSNAP_FRAME: /* snap to nearest frame (or second if drawing seconds) */
 					if (sipo->flag & SIPO_DRAWTIME)
-						td2d->loc[0] = (float)(floorf((td2d->loc[0]/secf) + 0.5f) * secf);
+						td2d->loc[0] = floorf((td2d->loc[0] / secf) + 0.5) * secf;
 					else
-						td2d->loc[0] = (float)(floorf(td2d->loc[0]+0.5f));
+						td2d->loc[0] = floorf(td2d->loc[0] + 0.5);
 					break;
 				
 				case SACTSNAP_MARKER: /* snap to nearest marker */

Modified: trunk/blender/source/blender/editors/transform/transform_input.c
===================================================================
--- trunk/blender/source/blender/editors/transform/transform_input.c	2012-05-22 19:16:20 UTC (rev 46909)
+++ trunk/blender/source/blender/editors/transform/transform_input.c	2012-05-22 21:16:59 UTC (rev 46910)
@@ -206,10 +206,10 @@
 			mdx = mval[0] - data[2];
 			mdy = mval[1] - data[3];
 
-			distance = (length != 0.0) ? (mdx * dx + mdy * dy) / length: 0.0;
+			distance = (length != 0.0) ? (mdx * dx + mdy * dy) / length : 0.0;
 		}
 
-		output[0] = (length != 0.0) ? (double)(distance / length) : 0.0f;
+		output[0] = (length != 0.0) ? (double)(distance / length) : 0.0;
 	}
 }
 




More information about the Bf-blender-cvs mailing list