Intel C compiler warns on this assignment in this if(), probably
authorDiego Pettenò <flameeyes@gmail.com>
Thu, 2 Oct 2008 19:20:11 +0000 (19:20 +0000)
committerGuillaume Poirier <gpoirier@mplayerhq.hu>
Thu, 2 Oct 2008 19:20:11 +0000 (19:20 +0000)
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%

Originally committed as revision 15525 to svn://svn.ffmpeg.org/ffmpeg/trunk

libavcodec/wmaenc.c

index 4558947..195c270 100644 (file)
@@ -178,7 +178,8 @@ static int encode_block(WMACodecContext *s, float (*src_coefs)[BLOCK_MAX_SIZE],
     }
 
     for(ch = 0; ch < s->nb_channels; ch++) {
     }
 
     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);
         }
     }
             init_exp(s, ch, fixed_exp);
         }
     }