diff options
author | Campbell Barton <ideasman42@gmail.com> | 2012-12-21 06:36:07 +0400 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2012-12-21 06:36:07 +0400 |
commit | 8f262d04133669d1439b024d3e2615c72ecf0c58 (patch) | |
tree | d437144ef64f354257e5dfddd6dc8ec77f6f8448 /source | |
parent | f2f2b6153a2a818ca940a4df5b7dafc743ef2d2f (diff) |
correct ed_view3d_project__internal() was unnecessarily casting (float -> short -> float).
Diffstat (limited to 'source')
-rw-r--r-- | source/blender/editors/space_view3d/view3d_project.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/source/blender/editors/space_view3d/view3d_project.c b/source/blender/editors/space_view3d/view3d_project.c index 34b983f83df..6227a55a890 100644 --- a/source/blender/editors/space_view3d/view3d_project.c +++ b/source/blender/editors/space_view3d/view3d_project.c @@ -116,7 +116,7 @@ static eV3DProjStatus ed_view3d_project__internal(ARegion *ar, float perspmat[4][4], const int is_local, /* normally hidden */ const float co[3], float r_co[2], const eV3DProjTest flag) { - float fx, fy, vec4[4]; + float vec4[4]; /* check for bad flags */ BLI_assert((flag & V3D_PROJ_TEST_ALL) == flag); @@ -135,12 +135,12 @@ static eV3DProjStatus ed_view3d_project__internal(ARegion *ar, mul_m4_v4(perspmat, vec4); if (vec4[3] > (float)BL_NEAR_CLIP) { - fx = ((float)ar->winx / 2.0f) * (1.0f + vec4[0] / vec4[3]); + const float fx = ((float)ar->winx / 2.0f) * (1.0f + vec4[0] / vec4[3]); if (((flag & V3D_PROJ_TEST_CLIP_WIN) == 0) || (fx > 0 && fx < ar->winx)) { - fy = ((float)ar->winy / 2.0f) * (1.0f + vec4[1] / vec4[3]); + const float fy = ((float)ar->winy / 2.0f) * (1.0f + vec4[1] / vec4[3]); if (((flag & V3D_PROJ_TEST_CLIP_WIN) == 0) || (fy > 0.0f && fy < (float)ar->winy)) { - r_co[0] = (short)floor(fx); - r_co[1] = (short)floor(fy); + r_co[0] = floorf(fx); + r_co[1] = floorf(fy); } else { return V3D_PROJ_RET_CLIP_WIN; |