[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