diff options
author | Felicia Lim <flim@google.com> | 2016-10-27 02:48:38 +0300 |
---|---|---|
committer | Jean-Marc Valin <jmvalin@jmvalin.ca> | 2016-10-28 06:36:57 +0300 |
commit | 33698ee159ab6ab1de5ae86c62249d0c2ae4d8f7 (patch) | |
tree | d25adeec4176cdcf5f1673adcec721492c7e9609 | |
parent | 9962bf3d941c68c2e167cb4f4fe79eb32f98f2a2 (diff) |
Update bandwidth and opusCanSwitch before generating SILK DTX ToC
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
-rw-r--r-- | src/opus_encoder.c | 17 | ||||
-rw-r--r-- | tests/test_opus_encode.c | 6 |
2 files changed, 10 insertions, 13 deletions
diff --git a/src/opus_encoder.c b/src/opus_encoder.c index 79ef3f1f..5d959848 100644 --- a/src/opus_encoder.c +++ b/src/opus_encoder.c @@ -2070,13 +2070,7 @@ opus_int32 opus_encode_native(OpusEncoder *st, const opus_val16 *pcm, int frame_ RESTORE_STACK; return OPUS_INTERNAL_ERROR; } - if (nBytes==0) - { - st->rangeFinal = 0; - data[-1] = gen_toc(st->mode, st->Fs/frame_size, curr_bandwidth, st->stream_channels); - RESTORE_STACK; - return 1; - } + /* Extract SILK internal bandwidth for signaling in first byte */ if( st->mode == MODE_SILK_ONLY ) { if( st->silk_mode.internalSampleRate == 8000 ) { @@ -2091,6 +2085,15 @@ opus_int32 opus_encode_native(OpusEncoder *st, const opus_val16 *pcm, int frame_ } st->silk_mode.opusCanSwitch = st->silk_mode.switchReady && !st->nonfinal_frame; + + if (nBytes==0) + { + st->rangeFinal = 0; + data[-1] = gen_toc(st->mode, st->Fs/frame_size, curr_bandwidth, st->stream_channels); + RESTORE_STACK; + return 1; + } + /* FIXME: How do we allocate the redundancy for CBR? */ if (st->silk_mode.opusCanSwitch) { diff --git a/tests/test_opus_encode.c b/tests/test_opus_encode.c index a7439254..e61ecb73 100644 --- a/tests/test_opus_encode.c +++ b/tests/test_opus_encode.c @@ -236,12 +236,6 @@ void fuzz_encoder_settings(const int num_encoders, const int num_setting_changes int frame_size_enum = get_frame_size_enum(frame_size, sampling_rate); force_channel = IMIN(force_channel, num_channels); - /* Todo: remove when a fix is available for coding SILK in DTX mode for >60 ms. - * Currently, SILK may internally adjust the bandwidth leading to mismatching - * bandwidths within a packet. */ - if (frame_size_ms_x2 > 120) - dtx = 0; - sprintf(debug_info, "fuzz_encoder_settings: %d kHz, %d ch, application: %d, " "%d bps, force ch: %d, vbr: %d, vbr constraint: %d, complexity: %d, " |