Changeset 511 for trunk/libs/libmath

Timestamp:

Aug 25, 2018, 6:30:55 PM (6 years ago)

Author:

viala@…

Message:

[libmath] Add braces arround a misleading indent for loop.

Sometimes braces could save your mind.

k_rem_pio2.c: In function 'kernel_rem_pio2':
k_rem_pio2.c:174:6: warning: this 'for' clause does not guard... [-Wmisleading-indentation]

for(j=0,fw=0.0;j<=jx;j++) fw += x[j]*f[jx+i-j]; q[i] = fw;

k_rem_pio2.c:174:54: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'for'

for(j=0,fw=0.0;j<=jx;j++) fw += x[j]*f[jx+i-j]; q[i] = fw;

File:

• trunk/libs/libmath/k_rem_pio2.c (modified) (1 diff)

trunk/libs/libmath/k_rem_pio2.c

@@ -168 +168 @@
     /* set up f[0] to f[jx+jk] where f[jx+jk] = ipio2[jv+jk] */
         j = jv-jx; m = jx+jk;
-        for(i=0;i<=m;i++,j++) f[i] = (j<0)? zero : (double) ipio2[j];
+        for (i=0;i<=m;i++,j++) { f[i] = (j<0)? zero : (double) ipio2[j]; }
 
     /* compute q[0],q[1],...q[jk] */
-        for (i=0;i<=jk;i++) {
-            for(j=0,fw=0.0;j<=jx;j++) fw += x[j]*f[jx+i-j]; q[i] = fw;
+        for ( i = 0; i <= jk; i++ ) {
+                for ( j = 0, fw = 0.0; j <= jx; j++) {
+                        fw += x[j] * f[jx + i - j];
+                }
+                q[i] = fw;
         }