Saves about 60 bits/sec for 20ms frames.

During SILK->CELT transitions, we were delaying the mode change by
one frame, but only after having ensured that bandwidth and mode were
consistent. We now do the delaying earlier.

Repacketizer now seems to work, though it doesn't yet handle
unmergeable packets. Also, test_opis no longer attempt to do proper
gapless at the end of the file, which was causing problems for
repacketization, but also with 32 bit overflows

Still more bugs to find

Thanks to Tim Terriberry for pointing out the issue.

This better distinguishes the reference implementation,
which is what this string is meant to do. Thanks to
Greg Maxwell for the suggestion.

This lets us verify this part of the API is working.

Remove trailing whitespace, convert tabs to 4 spaces, re-align
test_opus to use 4-space indents instead of 3, and re-wrap some
long lines.

This lets users print it out without context, which is a little
easier for clients.

The new opus_get_version_string() call just returns a static string
defined by OPUS_VERSION, which is passed in from the build system
through config.h (or a custom compile line). Provide a fallback to
"unknown" if the build system fails to actually provide that definition.

This restores compilation with Makefile.draft.

In general, this means there will be builds out there with non-specific
version strings, since ports won't be forced to update the string. While
that's unfortunate, I think it's more valuable that the library be simple
to build.

Ralph Giles authored 13 years ago and Jean-Marc Valin committed 13 years ago

This removes the configurability of the OPUS_TEST_RANGE_CODER_STATE
consistency check, in favour of always running it. The performance
penalty is low, and it's an important check on correctness for the
reference encoder to provide.

Ralph Giles authored 13 years ago and Jean-Marc Valin committed 13 years ago

Fix a warning on gcc 4.6. 'internal_sampling_rate_Hz' was
replaced by 'bandwidth' in a10e8796 but the local variable
holding the setting wasn't removed.

Ralph Giles authored 13 years ago and Jean-Marc Valin committed 13 years ago

A comment in opus_decode() said, "bit 6 is ignored" when in
fact bit 6 is the padding flag, parsed in the previous block.

The comment, and the previous one, are modified to reflect
the current code.

Also fixes two stereo mode switching issues.