diff options
| author | Diego Pettenò <flameeyes@gmail.com> | 2008-10-02 19:20:11 +0000 |
|---|---|---|
| committer | Guillaume Poirier <gpoirier@mplayerhq.hu> | 2008-10-02 19:20:11 +0000 |
| commit | 2d5174fc46de0cc6452290b901e0a68b729a6704 (patch) | |
| tree | 026e10d90353d05294f1d10ba94eafc59d72b432 | |
| parent | d3b9e7f19b88e02cd0625167cd1c303f23502604 (diff) | |
| download | ffmpeg-2d5174fc46de0cc6452290b901e0a68b729a6704.tar.gz | |
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
| -rw-r--r-- | libavcodec/wmaenc.c | 3 |
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); } } |