Fix side-channel index for VAD/LBRR flags.
authorTimothy B. Terriberry <tterribe@xiph.org>
Fri, 14 Oct 2011 23:14:36 +0000 (16:14 -0700)
committerJean-Marc Valin <jmvalin@jmvalin.ca>
Mon, 17 Oct 2011 20:03:41 +0000 (16:03 -0400)
commit1e03a6eb0472cc78e33541af11733f4f27ed3219
tree55e7478f507e7d803ae1ba577d65bc935015e5b4
parent53cc1a033a3183c9d137b03446db58d2e58c0e0a
Fix side-channel index for VAD/LBRR flags.

Neither the encoder nor decoder were incrementing the side-channel
 index for a mid-only frame.
Unfortunately, they used that index to index the VAD flags and LBRR
 flags.
This made the VAD flags for the side channel particularly useless,
 as you couldn't tell which frame a flag belonged to without
 decoding most of the packet.
It also desynched the LBRR information, as look at the wrong LBRR
 flags to decide when it had to code a mid-only flag.
If some frames were skipped in the side channel, then the last few
 VAD flags and LBRR flags would be garbage, but still get encoded.

This patch fixes this by continuing to increment nFramesDecoded or
 nFramesEncoded, even when skipping a frame in the side channel.
This makes the side-channel VAD and LBRR flags correspond to the
 correct time periods for frames greater than 20 ms.
It also fixes a bug where if DTX was not used on the packet where
 the side channel got turned off, it would never get used again
 until the encoder attempted to encode something in the side
 channel again.
silk/dec_API.c
silk/decode_frame.c
silk/enc_API.c
silk/fixed/encode_frame_FIX.c