[FFmpeg-cvslog] r15525 - trunk/libavcodec/wmaenc.c

[gpoirier]

Author: gpoirier
Date: Thu Oct  2 21:20:11 2008
New Revision: 15525

Log:
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%


Modified:
   trunk/libavcodec/wmaenc.c

Modified: trunk/libavcodec/wmaenc.c
==============================================================================
--- trunk/libavcodec/wmaenc.c	(original)
+++ trunk/libavcodec/wmaenc.c	Thu Oct  2 21:20:11 2008
@@ -178,7 +178,8 @@ static int encode_block(WMACodecContext 
     }
 
     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);
         }
     }