changeset 7985:5be50ba0ce24 libavcodec

Intel C compiler warns on this assignment in this if(), probably because it's being assigned as a constant. To avoid a spurious warning, split it into two instructions, which should also make it more logical once the FIXME is resolved. patch by Diego 'Flameeyes' Petten %flameeyes A gmail P com%
author gpoirier
date Thu, 02 Oct 2008 19:20:11 +0000
parents 24f2b8cc7918
children 38d577c3cb60
files wmaenc.c
diffstat 1 files changed, 2 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/wmaenc.c	Thu Oct 02 19:09:56 2008 +0000
+++ b/wmaenc.c	Thu Oct 02 19:20:11 2008 +0000
@@ -178,7 +178,8 @@
     }
 
     for(ch = 0; ch < s->nb_channels; ch++) {
-        if ((s->channel_coded[ch]= 1)) { //FIXME only set channel_coded when needed, instead of always
+        s->channel_coded[ch] = 1; //FIXME only set channel_coded when needed, instead of always
+        if (s->channel_coded[ch]) {
             init_exp(s, ch, fixed_exp);
         }
     }