summaryrefslogtreecommitdiff
path: root/libavcodec/wmaenc.c
diff options
context:
space:
mode:
authorDiego Pettenò <flameeyes@gmail.com>2008-10-02 19:20:11 +0000
committerGuillaume Poirier <gpoirier@mplayerhq.hu>2008-10-02 19:20:11 +0000
commit2d5174fc46de0cc6452290b901e0a68b729a6704 (patch)
tree026e10d90353d05294f1d10ba94eafc59d72b432 /libavcodec/wmaenc.c
parentd3b9e7f19b88e02cd0625167cd1c303f23502604 (diff)
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% Originally committed as revision 15525 to svn://svn.ffmpeg.org/ffmpeg/trunk
Diffstat (limited to 'libavcodec/wmaenc.c')
-rw-r--r--libavcodec/wmaenc.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/libavcodec/wmaenc.c b/libavcodec/wmaenc.c
index 4558947cb3..195c2702c6 100644
--- a/libavcodec/wmaenc.c
+++ b/libavcodec/wmaenc.c
@@ -178,7 +178,8 @@ static int encode_block(WMACodecContext *s, float (*src_coefs)[BLOCK_MAX_SIZE],
}
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);
}
}