Code Cleanup: correct fabsf/fabs/abs use
authorCampbell Barton <ideasman42@gmail.com>
Tue, 3 Dec 2013 22:23:29 +0000 (09:23 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Tue, 3 Dec 2013 22:24:38 +0000 (09:24 +1100)
source/blender/blenlib/BLI_math_vector.h
source/blender/blenlib/intern/math_matrix.c
source/blender/blenlib/intern/math_vector_inline.c
source/blender/editors/screen/area.c
source/blender/render/intern/source/shadeoutput.c

index 14445173ae64c101355730c507c6aed2397d324e..21bafef8678f439567080941aa462cd20e6e56ef 100644 (file)
@@ -144,6 +144,7 @@ MINLINE void negate_v3_short(short r[3]);
 
 MINLINE float dot_v2v2(const float a[2], const float b[2]) ATTR_WARN_UNUSED_RESULT;
 MINLINE float dot_v3v3(const float a[3], const float b[3]) ATTR_WARN_UNUSED_RESULT;
+MINLINE float dot_v4v4(const float a[4], const float b[4]) ATTR_WARN_UNUSED_RESULT;
 
 MINLINE float cross_v2v2(const float a[2], const float b[2]) ATTR_WARN_UNUSED_RESULT;
 MINLINE void cross_v3_v3v3(float r[3], const float a[3], const float b[3]);
index 1f52caac8e9f24df1495e80dcdf85d6bb5cfe8bb..5b1f924fd920c22ce43396cfdc866d283742552c 100644 (file)
@@ -988,7 +988,7 @@ bool is_orthogonal_m4(float m[4][4])
 
        for (i = 0; i < 4; i++) {
                for (j = 0; j < i; j++) {
-                       if (fabsf(dot_vn_vn(m[i], m[j], 4)) > 1.5f * FLT_EPSILON)
+                       if (fabsf(dot_v4v4(m[i], m[j])) > 1.5f * FLT_EPSILON)
                                return 0;
                }
 
@@ -1018,7 +1018,7 @@ bool is_orthonormal_m4(float m[4][4])
                int i;
 
                for (i = 0; i < 4; i++)
-                       if (fabsf(dot_vn_vn(m[i], m[i], 4) - 1) > 1.5f * FLT_EPSILON)
+                       if (fabsf(dot_v4v4(m[i], m[i]) - 1) > 1.5f * FLT_EPSILON)
                                return 0;
 
                return 1;
index cdbbcc5e0254012ce28344dfc6b9eb6a17aa3236..05be8bda709b50c5cd6c736dfdfcdf7e02097947 100644 (file)
@@ -595,6 +595,11 @@ MINLINE float dot_v3v3(const float a[3], const float b[3])
        return a[0] * b[0] + a[1] * b[1] + a[2] * b[2];
 }
 
+MINLINE float dot_v4v4(const float a[4], const float b[4])
+{
+       return a[0] * b[0] + a[1] * b[1] + a[2] * b[2] + a[3] * b[3];
+}
+
 MINLINE float cross_v2v2(const float a[2], const float b[2])
 {
        return a[0] * b[1] - a[1] * b[0];
index 8b89c2f0553b6edf37467d1f6d692043b9d96eb4..2caf318379bfd89b07cc3f89b70e0a080e603648 100644 (file)
@@ -160,8 +160,8 @@ static void area_draw_azone(short x1, short y1, short x2, short y2)
        int dx = x2 - x1;
        int dy = y2 - y1;
 
-       dx = copysign(ceilf(0.3f * fabsf(dx)), dx);
-       dy = copysign(ceilf(0.3f * fabsf(dy)), dy);
+       dx = copysign(ceilf(0.3f * abs(dx)), dx);
+       dy = copysign(ceilf(0.3f * abs(dy)), dy);
 
        glEnable(GL_BLEND);
        glEnable(GL_LINE_SMOOTH);
index 40a5a5d9a0584a9e428a4d51f9a046f132272d97..6cb34a67f45444cebac4b211a243d55a6ea94712 100644 (file)
@@ -228,7 +228,7 @@ static void spothalo(struct LampRen *lar, ShadeInput *shi, float *intens)
                maxz*= lar->sh_zfac;
                maxy= lar->imat[0][1]*p1[0]+lar->imat[1][1]*p1[1]+lar->imat[2][1]*p1[2];
 
-               if (fabsf(nray[2]) < FLT_EPSILON) {
+               if (fabs(nray[2]) < FLT_EPSILON) {
                        use_yco = TRUE;
                }
        }