From 2d5174fc46de0cc6452290b901e0a68b729a6704 Mon Sep 17 00:00:00 2001 From: Diego Pettenò Date: Thu, 2 Oct 2008 19:20:11 +0000 Subject: 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% MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Originally committed as revision 15525 to svn://svn.ffmpeg.org/ffmpeg/trunk --- libavcodec/wmaenc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'libavcodec/wmaenc.c') 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); } } -- cgit v1.2.3