friedman: Fix GCC warning about uninitialized fr.w.
authorBen Pfaff <blp@cs.stanford.edu>
Sun, 12 Dec 2010 05:09:17 +0000 (21:09 -0800)
committerBen Pfaff <blp@cs.stanford.edu>
Sun, 12 Dec 2010 06:10:45 +0000 (22:10 -0800)
On my machine GCC 4.4 complains:
friedman.c: In function ‘friedman_execute’:
friedman.c:106: warning: ‘fr.w’ may be used uninitialized in this function

This appears to be because GCC can't see that the assignment to fr.w and
its later use (in show_sig_box()) both have the same condition.  At any
rate, this commit suppresses the warning by always initializing fr.w.

src/language/stats/friedman.c

index 9711167b1d3a0ef8fea3d8600c614f4375873ea5..9950543f706c3c3ef16fedc179881b0767d87670 100644 (file)
@@ -202,6 +202,8 @@ friedman_execute (const struct dataset *ds,
       fr.w /= pow2 (fr.cc) * (pow3 (ost->n_vars) - ost->n_vars)
        - fr.cc * sigma_t;
     }
+  else
+    fr.w = SYSMIS;
 
   show_ranks_box (ost, &fr);
   show_sig_box (ost, &fr);