Fix NEON optimizations buffer read overrun

Thanks to Ray Essick

Avoid processing LPC coeffs beyond the given order in NEON optimizations

Fix typo in _FORTIFY_SOURCE define.

Signed-off-by: Ralph Giles <giles@thaumas.net>

Silk CNG adapts faster to received packets with lower gains

Add mising cmake file from make dist

Thanks to theislander on HA for reporting this.

Update checksums

Bump LT version

Adding the OPUS_GET_IN_DTX() query

CMake changes

If CMAKE_BUILD_TYPE is empty and CFlags are set then only use CFlags.
If None are set then use CMAKE_BUILD_TYPE by Release by default.

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

CMake changes

Make release build default
Made CUSTOM_MODE an option with default off
Added missing buildflags for Linux and security.

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

API for checking whether the encoder is in DTX

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

Correctly enable/disable SILK DTX with forced mode

Fix build errors using Makefile.mips

Broken by earlier ARM optimizations.

Adding CMake support for Windows, Mac, Linux and Android Co-Authored-By: evpobr

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

Fix #elif OPUS_ARM_INLINE_EDSP

Reported in https://github.com/xiph/opus/issues/108

Fix mismatching arg names in function declaration and definition

test_opus_encode: Use fprintf instead of snprintf

snprintf is not in C89.  Fixes opus-pedantic test failure introduced
in 59f8e5e4f.

Reset nb_no_activity_frames when analysis DTX isn't used

Improve silence handling

Copy the previous analysis data instead of flagging as invalid

Fixing initialization issues on small frame sizes

Fixes analysis buffering for silence and complexity changes

The previous code would go out of sync in those cases.

Avoiding problems with x87

logE[b] can end up slightly larger than tonal->highE[b] due to rounding

Properly handle a bad stream_id in OPUS_MULTISTREAM_GET_*_STATE_REQUEST

Thanks to Dmitriy for reporting this.

Fix FIXED_POINT conditional check

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

Clean up resources in projection test

Refactoring: Isolating the matrix-vector product in gemm_accum()

missing script

Adding the scripts used to train the RNN classifier

Sorry, no doc for now

Trivial sprintf to snprintf conversion. Some linkers warn about unbounded sprintf calls.

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

Make float2int() static with VS

Addresses the rest of https://github.com/xiph/opus/pull/79

Remove useless OPUS_SET_EXPERT_FRAME_DURATION() on CELT

frame diration is only handled at the Opus layer. There's no change
in behaviour, just less puzzling code.

Thanks to Dmitriy <dmilvdv@gmail.com> for reporting this.

Fixes misleading initialization with not enough zeros

update checksums

Bump LT version numbers

We're adding the ambisonics API

Proper detection of _mm_cvtss_si32 for MS compiler

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

Renaming compute_allocation to clt_compute_allocation() to avoid symbol clashes

As suggested in https://github.com/xiph/opus/pull/83

Prevent the SILK counter from overflowing after 2 years and 9 months

or "just" 1 year and 4 months for 10-ms frames.
The overflow can eventually cause a divide-by-zero when counter == -16

Thanks to Dmitry Malinin for reporting the bug.

Retrain RNN classifier weights to include reverberated speech

Fixes packet parsing for 16-bit CPUs

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.

Clarify configure --help

Update ISO Base Media Format draft to version 0.8.1.

- Switch to 'Opus' file type identification.
- Revise channel mapping to better support ambisonics.

Avoiding get_demixing_matrix() symbol clash on unified build

OpusMSDecoder does not have an arch field

Add include/opus_projection.h to opus_headers.mk

Apply equal bit allocation to ambisonic channels

Fixes issue #95 on GitHub.

Signed-off-by: Felicia Lim <flim@google.com>

Work around VS2015 internal compiler error

The error was:
c:\projects\opus\celt\celt_encoder.c(1019): fatal error C1001: An internal error has occurred in the compiler. [C:\projects\opus\win32\VS2015\opus.vcxproj]
  (compiler file 'f:\dd\vctools\compiler\utc\src\p2\main.c', line 255)
   To work around this problem, try simplifying or changing the program near the locations listed above.
  Please choose the Technical Support command on the Visual C++
   Help menu, or open the Technical Support help file for more information

  Internal Compiler Error in C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\bin\x86_amd64\CL.exe.  You will be prompted to send an error report to Microsoft later.

Fixing arithmetic problems for 16-bit CPUs in ambisonics_rate_allocation()

Fix bitrate allocation for channel mapping 2

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

win32: Add ambisonics sources to VS project

[mark.hsj: Add other missing files and update filter file]
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
Fixes https://github.com/xiph/libopusenc/issues/9
Fixes https://github.com/xiph/opus-tools/issues/32

Fixes C90 "mixed declarations and code" error

Remove ambisonics experimental flag

Enable ambisonics by default

Enable hardening by default

Fix comma that should have been a semicolon

Didn't cause any problem, but still good to fix. Reported by
keithchugg on github.

oops

Use ambisonics families 2 and 3 instead of 254 and 253

Validate multistream/projection decoder frame_size

Silence compiler warnings

clang -Wcast-align warnings with ambisonics enabled
clang -Wnull-pointer-arithmetic warnings in test_opus_api.c
gcc -Wimplicit-fallthrough warnings on arm
msvc warning C4244 in celt_encoder.c with fixed point

Make bandwidth detection more conservative to avoid low-passing frames

update checksums

Avoiding arithmetic on NULL pointer

Causes a warning with -Wnull-pointer-arithmetic and the code wasn't doing
anything anyway (the ctl call ignores NULLs)

Avoiding leaks when opus_demo exits with an error

This gets rid of false positives in static analysis
(see https://trac.webkit.org/changeset/232236/webkit)

Fix DISABLE_FLOAT_API unused parameter warning

Only call isqrt32() with a positive argument

Fixes test_opus_projection failure under ubsan, due to clz(0).

Switch from narrowband to wideband at 9 kb/s, don't use mediumband

Fix decision thresholds for larger frame sizes

Don't assume that frames larger than 20 ms save any overhead compared to 20 ms.

Fixes a SILK bandwidth switching regression

The bug was triggered because f982b84d started using prefill for
SILK bandwidth changes, which reinitialized the encoder state and
prevented the variable lowpass from working properly. To fix the problem,
we preserve the sampling rate and variable low-pass when prefilling.

Improving rate control for low bitrate

Using a finer table for the rate to SNR curves in silk_control_SNR().
It's now possible to have an SNR that reaches 0, so we can lower bitrate
down to ~5 kbps for narrowband and 5.5 kbps for wideband.

Oops, fix NaN test

Aborting on NaN in CELT

NaNs should be filtered at the Opus layer, so if there are any in the CELT
encoder, then it's likely something went horribly wrong (e.g. corrupted state).
In that case, better abort than have something bad happen.

Silk makes use of Opus VAD

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

README: Update to reflect RFC 7845 and 8251

Fix opus_packet_parse() non-null arg attribute

It is the size array that must be non-null.

two more asserts I forgot

Misc assertion cleanup

Harden SILK x86 code

Hardening silk/fixed

Hardening asserts for SILK float

Converting some silk_assert()s into hardening celt_assert()s

Only converted the ones that are really sure (not signal-dependent)
and that shouldn't add much run-time complexity

Some missing checks

Adding multistream decoder validation

Asserting on some ctl() calls that should never fail

Adds Opus decoder state validation

Adding decoder state validation

Fixing no-redundancy CELT->SILK and CELT->hybrid transitions

We make sure the CELT PLC we do in the transition uses the same bandwidth
as the previous (CELT) packet and not the new bandwidth

Adding ENABLE_HARDENING

Enables "safes" assertions even with ENABLE_ASSERTIONS isn't set

Support for Ambisonics.

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

Using a first-order filter for DC rejection

A second-order DC rejection filter is uselsss unless we have complex
poles. However, complex poles means we have to compute the filter as a
single pass (rather than two casdaded first-order filters), which has
numerical issues that would require a higher complexity to solve.
So rather than waste cycles with a second-order filter (with a longer
impulse response), we just go with a first-order filter.

Don't enable -fstack-protector-strong on Windows for now

It adds a libssp-0.dll dependency and prevents static linking

Adding -D_FORTIFY_SOURCE=2 when possible

Fixed off-by-one issue in Silk DTX

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

Enable -fstack-protector-strong by default on x86

The size overhead seems to be about 1% and the speed overhead is
"in the noise" (<2%).
The automake code is copied from opus-tools

Fix comment to take into account previous commit

Increasing the CELT bit allocation slightly for stereo hybrid

Fixes a "regression" from 1.2 to 1.3-beta that was especially noticeable
on a Korean speech sample. (it wasn't a real regression since 1.2 only
worked because it was using CELT)

Fix divide-by-zeros in opus_demo stats code

Fixes integer overflow in SILK VAD for 10-ms frames

Reported by Chandrakala Madhira on the mailing list

Forgot one RFC8251 behaviour to disable with --disable-rfc8251

Reported by daddesio in #opus

Disabling weak transients and 5ms resolution on voiced frames

Add a simple masking model to the spreading decision

This improves cases where a whole region is dominated by a handful of tones

Making sure importance[] is initialized even when we don't use dynalloc

oops