shithub: opus

Info  •  Files  •  Log  •  Branches

Fixes an internal error in the Opus encoder

The encoder would reserve bytes for redundancy and then decide to use
CELT, but not use those bytes.

--- a/src/opus_encoder.c

+++ b/src/opus_encoder.c

@@ -1729,6 +1729,13 @@

         st->mode = MODE_HYBRID;

     if (st->mode == MODE_HYBRID && curr_bandwidth <= OPUS_BANDWIDTH_WIDEBAND)

         st->mode = MODE_SILK_ONLY;

+    /* If we decided to go with CELT, make sure redundancy is off, no matter what

+       we decided earlier. */

+    if (st->mode == MODE_CELT_ONLY)

+    {

+        redundancy = 0;

+        redundancy_bytes = 0;

+    }

     /* printf("%d %d %d %d\n", st->bitrate_bps, st->stream_channels, st->mode, curr_bandwidth); */

     bytes_target = IMIN(max_data_bytes-redundancy_bytes, st->bitrate_bps * frame_size / (st->Fs * 8)) - 1;