Age | Commit message (Collapse) | Author |
|
Without that change, a very long (> 682 ms) illegal packet could trigger
a wrap-around in the test and be accepted as valid.
Only 16-bit architectures (e.g. TI C5x) were affected.
|
|
- celt/modes.c:430:14: warning: cast from 'const unsigned char *' to
'opus_int16 *' increases required alignment from 1 to 2 [-Wcast-align]
- 'C[0][1]' may be used uninitialized [-Wmaybe-uninitialized]
- Unused variable/parameter
- Value stored is never read
- MSVC warnings about "possible loss of data" due to type conversions
- MSVC warning C4146: unary minus operator applied to unsigned type
- silk/NLSF_del_dec_quant.c:137:20: warning: array subscript is above
array bounds [-Warray-bounds] (gcc -O3 false positive)
- src/mlp_train.h:39:20: warning: function declaration isn't a prototype
[-Wstrict-prototypes]
- Remove SMALL_FOOTPRINT code from SSE 4.1 FIR implementation, matching
the C implementation.
The clang -Wcast-align warnings with SSE intrinsics are a known
clang issue: https://llvm.org/bugs/show_bug.cgi?id=20670
|
|
No values outside of +/-1 detected now.
|
|
-ffast-math
|
|
opus_packet_parse_impl()
len<0 still returns OPUS_BAD_ARG
|
|
Rather than read invalid memory
|
|
|
|
|
|
Because it's indirectly used in the encoder (through the repackerizer).
|
|
opus_decode() and opus_multistream_decode() now apply soft clipping
before converting to 16-bit int. This should produce better a higher
quality result than hard clipping like we were doing before. The _float()
API isn't affected, but the clipping function is exported so users can
manually apply the soft clipping.
|
|
Also added 3rd clause to "master" COPYING file
|
|
In f9265ddf25cc opus_get_version_string() was moved
to libcelt, but the fallback definition for OPUS_VERSION
stayed in src/opus.c. This is helpful for those using
and external build system, which may not duplicate
all the defines from the included build files.
|
|
|
|
|
|
|
|
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 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.
|
|
|
|
|
|
|