]> granicus.if.org Git - graphviz/commitdiff
neatogen: [nfc] remove some temporaries
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Tue, 15 Feb 2022 08:35:30 +0000 (19:35 +1100)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 16 Feb 2022 10:09:59 +0000 (21:09 +1100)
These seemed to be making the code less clear rather than more clear.

lib/neatogen/matrix_ops.c

index ad33e0dd04fcdba8efb42484aeb4889ab439d732..e123c0184a64ae6e94872cd8c9c46e2c6dbf7af8 100644 (file)
@@ -599,10 +599,9 @@ void square_vec(int n, float *vec)
 void invert_vec(int n, float *vec)
 {
     int i;
-    float v;
     for (i = 0; i < n; i++) {
-       if ((v = vec[i]) != 0.0)
-           vec[i] = 1.0f / v;
+       if (vec[i] != 0.0)
+           vec[i] = 1.0f / vec[i];
     }
 }
 
@@ -617,10 +616,9 @@ void sqrt_vec(int n, float *vec)
 void sqrt_vecf(int n, float *source, float *target)
 {
     int i;
-    float v;
     for (i = 0; i < n; i++) {
-       if ((v = source[i]) >= 0.0) {
-           target[i] = sqrtf(v);
+       if (source[i] >= 0.0) {
+           target[i] = sqrtf(source[i]);
        }
     }
 }
@@ -628,10 +626,9 @@ void sqrt_vecf(int n, float *source, float *target)
 void invert_sqrt_vec(int n, float *vec)
 {
     int i;
-    float v;
     for (i = 0; i < n; i++) {
-       if ((v = vec[i]) > 0.0) {
-           vec[i] = 1.0f / sqrtf(v);
+       if (vec[i] > 0.0) {
+           vec[i] = 1.0f / sqrtf(vec[i]);
        }
     }
 }